Implement the 'seen comments' functionality (red dot) for the comments [SCI-5253]

app/helpers/comment_helper.rb

@@ -254,4 +254,8 @@ module CommentHelper
             project: result.my_module.experiment.project,
             message_items: { result: result.id })
   end
+
+  def has_unseen_comments?(commentable)
+    commentable.comments.where('? = ANY (unseen_by)', current_user.id).any?
+  end
 end

app/models/comment.rb

@@ -60,4 +60,10 @@ class Comment < ApplicationRecord
         .offset((page - 1) * Constants::SEARCH_LIMIT)
     end
   end
+
+  def self.mark_as_seen_by(user)
+    # rubocop:disable Rails/SkipsModelValidations
+    all.where('? = ANY (unseen_by)', user.id).update_all("unseen_by = array_remove(unseen_by, #{user.id.to_i}::bigint)")
+    # rubocop:enable Rails/SkipsModelValidations
+  end
 end

app/models/my_module.rb

@@ -249,7 +249,7 @@ class MyModule < ApplicationRecord
                           .where('comments.id <  ?', last_id)
                           .order(created_at: :desc)
                           .limit(per_page)
-    comments.reverse
+    TaskComment.from(comments, :comments).order(created_at: :asc)
   end
 
   def last_activities(last_id = 1,

app/models/project.rb

@@ -164,7 +164,7 @@ class Project < ApplicationRecord
                              .where('comments.id <  ?', last_id)
                              .order(created_at: :desc)
                              .limit(per_page)
-    comments.reverse
+    ProjectComment.from(comments, :comments).order(created_at: :asc)
   end
 
   def unassigned_users

app/models/project_comment.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 class ProjectComment < Comment
+  before_create :fill_unseen_by
+
   belongs_to :project, foreign_key: :associated_id, inverse_of: :project_comments, touch: true
 
   validates :project, presence: true
@@ -8,4 +10,10 @@ class ProjectComment < Comment
   def commentable
     project
   end
+
+  private
+
+  def fill_unseen_by
+    self.unseen_by += project.users.where.not(id: user.id).pluck(:id)
+  end
 end

app/models/result.rb

@@ -76,7 +76,7 @@ class Result < ApplicationRecord
                             .where('comments.id <  ?', last_id)
                             .order(created_at: :desc)
                             .limit(per_page)
-    comments.reverse
+    ResultComment.from(comments, :comments).order(created_at: :asc)
   end
 
   def is_text

app/models/result_comment.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 class ResultComment < Comment
+  before_create :fill_unseen_by
+
   belongs_to :result, foreign_key: :associated_id, inverse_of: :result_comments, touch: true
 
   validates :result, presence: true
@@ -8,4 +10,10 @@ class ResultComment < Comment
   def commentable
     result
   end
+
+  private
+
+  def fill_unseen_by
+    self.unseen_by += result.my_module.experiment.project.users.where.not(id: user.id).pluck(:id)
+  end
 end

app/models/step.rb

@@ -102,7 +102,7 @@ class Step < ApplicationRecord
                           .where('comments.id <  ?', last_id)
                           .order(created_at: :desc)
                           .limit(per_page)
-    comments.reverse
+    StepComment.from(comments, :comments).order(created_at: :asc)
   end
 
   def save(current_user=nil)

app/models/step_comment.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 class StepComment < Comment
+  before_create :fill_unseen_by
+
   belongs_to :step, foreign_key: :associated_id, inverse_of: :step_comments, touch: true
 
   validates :step, presence: true
@@ -8,4 +10,10 @@ class StepComment < Comment
   def commentable
     step
   end
+
+  private
+
+  def fill_unseen_by
+    self.unseen_by += step.protocol.my_module.experiment.project.users.where.not(id: user.id).pluck(:id)
+  end
 end

app/models/task_comment.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
 class TaskComment < Comment
+  before_create :fill_unseen_by
+
   belongs_to :my_module, foreign_key: :associated_id, inverse_of: :task_comments
 
   validates :my_module, presence: true
@@ -8,4 +10,10 @@ class TaskComment < Comment
   def commentable
     my_module
   end
+
+  private
+
+  def fill_unseen_by
+    self.unseen_by += my_module.experiment.project.users.where.not(id: user.id).pluck(:id)
+  end
 end

app/views/projects/index/_project_actions_dropdown.html.erb

@@ -1,7 +1,7 @@
 <% cache [current_user, project] do %>
   <% active = !project.archived %>
   <% if (active && (can_manage_project?(project) || can_archive_project?(project))) || (!active && can_restore_project?(project)) %>
-    <div class="dropdown project-actions-menu new-comments">
+    <div class="dropdown project-actions-menu <%= 'new-comments' if has_unseen_comments?(project) %>">
       <button class="btn btn-light dropdown-toggle icon-btn" type="button" id="projectActionsDropdown" data-toggle="dropdown" aria-haspopup="true" aria-expanded="true">
         <i class="fas fa-ellipsis-h"></i>
       </button>

app/views/shared/comments/_comments.html.erb

@@ -1,4 +1,4 @@
-<div class="comments-container" data-object-id = <%= object.id %>> 
+<div class="comments-container" data-object-id = <%= object.id %>>
   <% per_page = Constants::COMMENTS_SEARCH_LIMIT %>
   <div class="content-comments inline_scroll_block">
     <% if comments.size == per_page %>
@@ -12,15 +12,16 @@
     <% end %>
     <div class="comments-list">
       <%= render partial: 'shared/comments/list.html.erb', locals: { comments: comments} %>
+      <% comments.mark_as_seen_by(current_user) %>
     </div>
   </div>
   <% if can_create_comments %>
   <div class="new-message-container" data-create-url="<%= create_url %>">
-    <%= text_area_tag 'message', '',  placeholder: t('general.comment_placeholder_new'), 
+    <%= text_area_tag 'message', '',  placeholder: t('general.comment_placeholder_new'),
-                                        class: 'form-control  smart-text-area textarea-sm', 
+                                      class: 'form-control  smart-text-area textarea-sm',
-                                        data: { 'atwho-edit' => '' } %>
+                                      data: { 'atwho-edit' => '' } %>
     <i class="fas fa-paper-plane new-comment-button"></i>
     <span class="new-message-error"></span>
   </div>
   <% end %>
-</div>
+</div>

db/migrate/20201126203713_add_unseen_by_to_comments.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddUnseenByToComments < ActiveRecord::Migration[6.0]
+  def change
+    add_column :comments, :unseen_by, :bigint, array: true, default: []
+  end
+end

db/structure.sql

@@ -332,7 +332,8 @@ CREATE TABLE public.comments (
     updated_at timestamp without time zone NOT NULL,
     last_modified_by_id bigint,
     type character varying,
-    associated_id integer
+    associated_id integer,
+    unseen_by bigint[] DEFAULT '{}'::bigint[]
 );
 
 
@@ -7062,6 +7063,7 @@ INSERT INTO "schema_migrations" (version) VALUES
 ('20200714082503'),
 ('20200826143431'),
 ('20200909121441'),
-('20201028103608');
+('20201028103608'),
+('20201126203713');