Fixed PartnerGroup not checking if reminder_schedule needs to be updates, added comments explaining need for the check

Author: Benjamin-Couey

app/models/organization.rb

@@ -94,6 +94,8 @@ def upcoming
   end
 
   before_save do
+    # To avoid constantly changing the start date of the reminder_schedule, only update the schedule if something has actually
+    # changed.
     if should_update_reminder_schedule
       self.reminder_schedule = create_schedule
     end

app/models/partner_group.rb

@@ -20,7 +20,11 @@ class PartnerGroup < ApplicationRecord
   has_and_belongs_to_many :item_categories
 
   before_save do
-    self.reminder_schedule = create_schedule
+    # To avoid constantly changing the start date of the reminder_schedule, only update the schedule if something has actually
+    # changed.
+    if should_update_reminder_schedule
+      self.reminder_schedule = create_schedule
+    end
   end
 
   validates :name, presence: true, uniqueness: { scope: :organization }