Allow comment deletion via nested attributes

app/controllers/organizations_controller.rb

@@ -199,7 +199,7 @@ def organization_params
         :end_date,
         :_destroy
       ],
-      comments_attributes: [ :id, :body ],
+      comments_attributes: [ :id, :body, :_destroy ],
       addresses_attributes: [
         :id,
         :address_type,

app/controllers/people_controller.rb

@@ -399,7 +399,7 @@ def person_params
         :end_date,
         :_destroy
       ],
-      comments_attributes: [ :id, :body ],
+      comments_attributes: [ :id, :body, :_destroy ],
     )
   end
 end

app/controllers/users_controller.rb

@@ -408,7 +408,7 @@ def user_params
       :phone, :phone2, :phone3, :birthday, :best_time_to_call, :notes, # legacy to remove later
       #####
 
-      comments_attributes: [ :id, :body ],
+      comments_attributes: [ :id, :body, :_destroy ],
       affiliations_attributes: [ :id, :organization_id, :position, :title, :inactive, :primary_contact, :start_date, :end_date, :_destroy ],
     )
   end

app/models/event_registration.rb

@@ -9,7 +9,7 @@ class EventRegistration < ApplicationRecord
 
   before_destroy :create_refund_payments
 
-  accepts_nested_attributes_for :comments, reject_if: proc { |attrs| attrs["body"].blank? }
+  accepts_nested_attributes_for :comments, allow_destroy: true, reject_if: proc { |attrs| attrs["body"].blank? }
 
   before_create :generate_slug
   after_create :snapshot_registrant_organizations

app/models/organization.rb

@@ -48,7 +48,7 @@ class Organization < ApplicationRecord
   after_save :remove_duplicate_sectorable_items
   accepts_nested_attributes_for :affiliations, allow_destroy: true,
                                 reject_if: proc { |attrs| attrs["person_id"].blank? }
-  accepts_nested_attributes_for :comments, reject_if: proc { |attrs| attrs["body"].blank? }
+  accepts_nested_attributes_for :comments, allow_destroy: true, reject_if: proc { |attrs| attrs["body"].blank? }
 
   # SearchCop
   include SearchCop

app/models/person.rb

@@ -62,7 +62,7 @@ class Person < ApplicationRecord
   accepts_nested_attributes_for :user, update_only: true
   accepts_nested_attributes_for :affiliations, allow_destroy: true,
     reject_if: proc { |attrs| attrs["organization_id"].blank? }
-  accepts_nested_attributes_for :comments, reject_if: proc { |attrs| attrs["body"].blank? }
+  accepts_nested_attributes_for :comments, allow_destroy: true, reject_if: proc { |attrs| attrs["body"].blank? }
 
   # Search Cop
   include SearchCop

app/models/user.rb

@@ -55,7 +55,7 @@ class User < ApplicationRecord
   has_many :user_form_form_fields, through: :user_forms, dependent: :destroy
   # Nested attributes
   accepts_nested_attributes_for :user_forms
-  accepts_nested_attributes_for :comments, reject_if: proc { |attrs| attrs["body"].blank? }
+  accepts_nested_attributes_for :comments, allow_destroy: true, reject_if: proc { |attrs| attrs["body"].blank? }
 
 
   before_validation :strip_whitespace

app/views/comments/_comment_fields.html.erb

@@ -71,6 +71,8 @@
                     as: :text,
                     input_html: { rows: 1, class: "w-full rounded-md border-gray-300 shadow-sm focus:border-blue-500 focus:ring focus:ring-blue-200 text-sm" } %>
       </div>
+      <%= link_to_remove_association "Remove", f,
+            class: "text-sm text-gray-400 hover:text-red-600 underline whitespace-nowrap mt-2 shrink-0" %>
     </div>
   <% else %>
     <% current_first_name = current_user&.person&.first_name || current_user&.name.to_s.split.first %>