1069: Comment what needs to be removed once the flag is retired

Author: danhalson

app/models/school.rb

@@ -42,6 +42,7 @@ class School < ApplicationRecord
 
   before_save :format_uk_postal_code, if: :should_format_uk_postal_code?
 
+  # TODO: Remove the conditional once the feature flag is retired
   after_commit :generate_code!, on: :create, if: -> { FeatureFlags.immediate_school_onboarding? }
 
   def self.find_for_user!(user)
@@ -64,6 +65,7 @@ def rejected?
   end
 
   def verify!
+    # TODO: Remove this line once the feature flag is retired
     generate_code! unless FeatureFlags.immediate_school_onboarding?
 
     update!(verified_at: Time.zone.now)

app/services/school_verification_service.rb

@@ -11,6 +11,7 @@ def verify(token: nil)
     School.transaction do
       school.verify!
 
+      # TODO: Remove this line once the feature flag is retired
       SchoolOnboardingService.new(school).onboard(token: token) unless FeatureFlags.immediate_school_onboarding?
     end
   rescue StandardError => e

lib/concepts/school/operations/create.rb

@@ -10,6 +10,7 @@ def call(school_params:, creator_id:, token:)
         School.transaction do
           response[:school].save!
 
+          # TODO: Remove this conditional once the feature flag is retired
           SchoolOnboardingService.new(response[:school]).onboard(token:) if FeatureFlags.immediate_school_onboarding?
         end