Standup comments 1

app/components/standup_meetings/update_component.html.erb

@@ -6,4 +6,8 @@
   <div class="divider my-1 "></div>
 
   <%= tag.span(standup_content, class: "grow whitespace-pre-line") %>
+
+  <%= link_to standup_meeting_sections_path(standup_meeting_id: @standup_meeting.id, section: @content_type), class: '' do %>
+    <i class="fas fa-comments"></i> Comments
+  <% end %>
 </div>

app/controllers/standup_meeting_comments_controller.rb

@@ -0,0 +1,39 @@
+class StandupMeetingCommentsController < ApplicationController
+  before_action :set_standup_meeting, only: %i[create]
+
+  def edit; end
+
+  def create
+    @standup_meeting_comment = @standup_meeting.standup_meeting_comments.new(standup_meeting_comment_params)
+    @standup_meeting_comment.user = current_user
+    if @standup_meeting_comment.save
+      redirect_to @standup_meeting_comment, notice: 'Comment was successfully created.'
+    else
+      render :new
+    end
+  end
+
+  def destroy
+    @standup_meeting_comment = StandupMeetingComment.find(params[:id])
+    authorize @standup_meeting_comment
+    respond_to do |format|
+      if @standup_meeting_comment.destroy
+        format.html do
+          redirect_back(fallback_location: root_path, notice: 'Comment was successfully deleted.')
+        end
+      else
+        format.html { render :show, status: :unprocessable_entity }
+      end
+    end
+  end
+
+  private
+
+  def set_standup_meeting
+    @standup_meeting = StandupMeeting.find(params[:standup_meeting_id])
+  end
+
+  def standup_meeting_comment_params
+    params.require(:standup_meeting_comment).permit(:content, :section_name, :standup_meeting_id)
+  end
+end

app/controllers/standup_meetings/sections_controller.rb

@@ -0,0 +1,13 @@
+class StandupMeetings::SectionsController < ApplicationController
+  def index
+    @standup_meeting = StandupMeeting.find(params[:standup_meeting_id])
+    @section = params[:section]
+
+    if @standup_meeting.allowed_section?(@section)
+      @question = @standup_meeting.send(@section)
+    else
+      flash[:alert] = 'Invalid section'
+      redirect_to standup_meeting_group_standup_meetings_path(@standup_meeting.group)
+    end
+  end
+end

app/controllers/standup_meetings_controller.rb

@@ -28,6 +28,7 @@ def index
     )
     @completed_meetings = @standup_meetings.filter(&:completed?)
   end
+
   # rubocop:enable Metrics/AbcSize
 
   def edit

app/helpers/standup_meeting_comments_helper.rb

@@ -0,0 +1,2 @@
+module StandupMeetingCommentsHelper
+end

app/models/standup_meeting.rb

@@ -23,6 +23,8 @@
 class StandupMeeting < ApplicationRecord
   has_noticed_notifications
 
+  has_many :standup_meeting_comments, dependent: :destroy
+
   belongs_to :standup_meeting_group, inverse_of: :standup_meetings
   belongs_to :user
 
@@ -36,6 +38,8 @@ class StandupMeeting < ApplicationRecord
 
   validates :meeting_date, presence: true
 
+  ALLOWED_SECTIONS = %w[yesterday_work_description today_work_description blockers_description].freeze
+
   enum status: {
     draft: 0,
     completed: 1,
@@ -44,4 +48,23 @@ class StandupMeeting < ApplicationRecord
   }
 
   scope :for_member, ->(user, group) { where(user:, standup_meeting_group: group) }
+
+  def comments(section)
+    standup_meeting_comments.where(section_name: section)
+  end
+
+  def section_content(section)
+    case section
+    when 'yesterday_work_description'
+      yesterday_work_description
+    when 'today_work_description'
+      today_work_description
+    when 'blockers_description'
+      blockers_description
+    end
+  end
+
+  def allowed_section?(secion_name)
+    ALLOWED_SECTIONS.include?(secion_name)
+  end
 end

app/models/standup_meeting_comment.rb

@@ -0,0 +1,27 @@
+# == Schema Information
+#
+# Table name: standup_meeting_comments
+#
+#  id                 :bigint           not null, primary key
+#  section_name       :string           not null
+#  created_at         :datetime         not null
+#  updated_at         :datetime         not null
+#  standup_meeting_id :bigint           not null
+#  user_id            :bigint           not null
+#
+# Indexes
+#
+#  index_standup_meeting_comments_on_standup_meeting_id  (standup_meeting_id)
+#  index_standup_meeting_comments_on_user_id             (user_id)
+#
+# Foreign Keys
+#
+#  fk_rails_...  (standup_meeting_id => standup_meetings.id)
+#  fk_rails_...  (user_id => users.id)
+#
+class StandupMeetingComment < ApplicationRecord
+  belongs_to :standup_meeting
+  belongs_to :user
+
+  has_rich_text :content
+end

app/models/user.rb

@@ -82,6 +82,8 @@ class User < ApplicationRecord
   has_one :current_resume, -> { where(current: true) }, class_name: 'Resume', dependent: nil
   # rubocop:enable Rails/InverseOf
 
+  has_many :standup_meeting_comments, dependent: :destroy
+
   validates :first_name, presence: true
   validates :last_name, presence: true
 

app/policies/standup_meeting_comment_policy.rb

@@ -0,0 +1,11 @@
+class StandupMeetingCommentPolicy < ApplicationPolicy
+  alias_method :standup_meeting_comment, :record
+
+  def destroy?
+    user == standup_meeting_comment.user
+  end
+
+  def edit?
+    user == standup_meeting_comment.user
+  end
+end

app/views/standup_meetings/sections/index.html.erb

@@ -0,0 +1,43 @@
+<h1 class="text-2xl font-bold mb-4 text-center "><%= @section.humanize.titleize %></h1>
+<div class="max-w-6xl mx-auto border rounded-lg p-4">
+  <h1 class="text-1xl font-bold mb-4"><%= @standup_meeting.section_content(@section) %> </h1>
+</div>
+
+<div class="max-w-6xl mx-auto mt-4">
+  <% @standup_meeting.comments(@section).each do |comment| %>
+    <div class="flex-1 border rounded-lg leading-relaxed mb-3">
+      <div class="bg-gray-100 px-3 py-1 rounded-t-lg">
+        <strong class="text-sm font-semibold"><%= comment.user.first_name %></strong> <span class="text-xs text-gray-500"><%= time_ago_in_words(comment.created_at) %></span>
+      </div>
+
+      <div class="mt-2 mx-2">
+        <p class="text-sm">
+          <%= comment.content %>
+        </p>
+      </div>
+
+      <div class="mt-4 flex justify-end m-2 text-xs text-gray-500">
+        <% if policy(comment).edit?  %>
+          <div class="text-sm hover:text-blue-600 cursor-pointer mr-2">Edit</div>
+        <% end %>
+        <% if policy(comment).destroy? %>
+          <div class="text-sm hover:text-blue-600 cursor-pointer"> <%= link_to 'Delete', standup_meeting_standup_meeting_comment_path(@standup_meeting, comment), method: :delete, data: { turbo_method: :delete } %></div>
+        <% end %>
+      </div>
+    </div>
+  <% end %>
+
+  <div class="border-t-4 border-gray-300 mt-4">
+    <div>Add a comment</div>
+    <%= form_with model:[@standup_meeting, StandupMeetingComment.new], html: { class: 'border rounded-lg mt-4 p-2' } do |f| %>
+      <%= f.hidden_field :section_name, value: @section %>
+
+      <div class="flex flex-col justify-between space-y-2 mb-4">
+        <%= f.rich_text_area :content, class:"textarea textarea-bordered" %>
+      </div>
+      <div class="form-control">
+        <%= f.submit class:"btn btn-primary btn-sm mx-auto"%>
+      </div>
+    <% end %>
+  </div>
+</div>

config/routes.rb

@@ -2,6 +2,7 @@
 require 'admin_constraint'
 
 Rails.application.routes.draw do
+  resources :standup_meeting_comments, only: %i[index show new create edit update]
   constraints(AdminConstraint) do
     mount Sidekiq::Web => '/sidekiq'
   end
@@ -70,5 +71,12 @@
     get :faq
   end
 
+  resources :standup_meetings do
+    resources :standup_meeting_comments, only: %i[create update destroy]
+    scope module: :standup_meetings do
+      resources :sections, only: %i[create show index]
+    end
+  end
+
   resources :rich_text_reactions, only: %i[create destroy]
 end

db/migrate/20231127185636_create_standup_meeting_comments.rb

@@ -0,0 +1,10 @@
+class CreateStandupMeetingComments < ActiveRecord::Migration[7.0]
+  def change
+    create_table :standup_meeting_comments do |t|
+      t.string :section_name, null: false
+      t.references :standup_meeting, null: false, foreign_key: true
+
+      t.timestamps
+    end
+  end
+end

db/migrate/20231201004804_add_user_to_standup_meeting_comments.rb

@@ -0,0 +1,5 @@
+class AddUserToStandupMeetingComments < ActiveRecord::Migration[7.0]
+  def change
+    add_reference :standup_meeting_comments, :user, null: false, foreign_key: true
+  end
+end

spec/components/standup_meetings/update_component_spec.rb

@@ -9,7 +9,7 @@
     let(:yesterday_work_description) { 'Lorem ipsum' }
     let(:today_work_description) { 'Sit amet' }
     let(:standup_meeting) do
-      build(:standup_meeting, yesterday_work_description:, today_work_description:)
+      create(:standup_meeting, yesterday_work_description:, today_work_description:)
     end
 
     it "renders yesterday's content when passed the yesterday content type" do

spec/factories/standup_meeting_comments.rb

@@ -0,0 +1,28 @@
+# == Schema Information
+#
+# Table name: standup_meeting_comments
+#
+#  id                 :bigint           not null, primary key
+#  section_name       :string           not null
+#  created_at         :datetime         not null
+#  updated_at         :datetime         not null
+#  standup_meeting_id :bigint           not null
+#  user_id            :bigint           not null
+#
+# Indexes
+#
+#  index_standup_meeting_comments_on_standup_meeting_id  (standup_meeting_id)
+#  index_standup_meeting_comments_on_user_id             (user_id)
+#
+# Foreign Keys
+#
+#  fk_rails_...  (standup_meeting_id => standup_meetings.id)
+#  fk_rails_...  (user_id => users.id)
+#
+FactoryBot.define do
+  factory :standup_meeting_comment do
+    section_name { 'MyString' }
+    standup_meeting
+    user
+  end
+end

spec/helpers/standup_meeting_comments_helper_spec.rb

@@ -0,0 +1,15 @@
+require 'rails_helper'
+
+# Specs in this file have access to a helper object that includes
+# the StandupMeetingCommentsHelper. For example:
+#
+# describe StandupMeetingCommentsHelper do
+#   describe "string concat" do
+#     it "concats two strings with spaces" do
+#       expect(helper.concat_strings("this","that")).to eq("this that")
+#     end
+#   end
+# end
+RSpec.describe StandupMeetingCommentsHelper do
+  pending "add some examples to (or delete) #{__FILE__}"
+end

spec/models/standup_meeting_comment_spec.rb

@@ -0,0 +1,26 @@
+# == Schema Information
+#
+# Table name: standup_meeting_comments
+#
+#  id                 :bigint           not null, primary key
+#  section_name       :string           not null
+#  created_at         :datetime         not null
+#  updated_at         :datetime         not null
+#  standup_meeting_id :bigint           not null
+#  user_id            :bigint           not null
+#
+# Indexes
+#
+#  index_standup_meeting_comments_on_standup_meeting_id  (standup_meeting_id)
+#  index_standup_meeting_comments_on_user_id             (user_id)
+#
+# Foreign Keys
+#
+#  fk_rails_...  (standup_meeting_id => standup_meetings.id)
+#  fk_rails_...  (user_id => users.id)
+#
+require 'rails_helper'
+
+RSpec.describe StandupMeetingComment do
+  pending "add some examples to (or delete) #{__FILE__}"
+end