Add Ats notice on sign in (#8330)

richardpattinson · web-flow · commit caf4bfe6ec64 · 2026-03-05T15:24:31.000Z
## Trello card URL https://trello.com/c/rsknc856/2367-hiring-staff-notice-upon-sign-in-for-ats-religious-app-forms ## Changes in this PR: We want to highlight our new ATS features and faith forms to hiring staff so want to advertise this on a service screen. Question: Should this screen only appear when first signed in + a flag should be added as to whether the user has accepted the screen - or should it be displayed everytime? ## Screenshots of UI changes: ### Before ### After ## Checklists: ### Data & Schema Changes If this PR modifies data structures or validations, check the following: - [ ] Adds/removes model validations - [ ] Adds/removes database fields - [ ] Modifies Vacancy enumerables (phases, working patterns, job roles, key stages, etc.) <details> <summary>If any of the above options has changed then the author must check/resolve all of the following...</summary> ### Integration Impact Does this change affect any of these integrations? - [ ] DfE Analytics platform - [ ] Legacy imports mappings - [ ] DWP Find a Job export mappings - [ ] Publisher ATS API (may require mapping updates or API versioning) ### User Experience & Data Integrity Could this change impact: - [ ] Existing subscription alerts (will legacy subscription search filters break?) - [ ] Legacy vacancy copying (will copied vacancies fail new validations?) - [ ] In-progress drafts for Vacancies or Job Applications </details>
diff --git a/app/controllers/publishers/ats_interstitials_controller.rb b/app/controllers/publishers/ats_interstitials_controller.rb
@@ -0,0 +1,18 @@
+class Publishers::AtsInterstitialsController < Publishers::BaseController
+  skip_before_action :check_ats_interstitial_acknowledged, only: %i[show update]
+
+  def show
+    @organisation = current_organisation
+    @variant = @organisation.ats_interstitial_variant
+  end
+
+  def update
+    if current_publisher.update(acknowledged_ats_and_religious_form_interstitial: true)
+      redirect_to organisation_jobs_with_type_path
+    else
+      @organisation = current_organisation
+      @variant = @organisation.ats_interstitial_variant
+      render :show
+    end
+  end
+end
diff --git a/app/controllers/publishers/base_controller.rb b/app/controllers/publishers/base_controller.rb
@@ -1,19 +1,20 @@
 module Publishers
-  class BaseController < ApplicationController
-    include ReturnPathTracking
+  class Publishers::BaseController < ApplicationController
     include LoginRequired
 
     before_action :check_terms_and_conditions
-    before_action :check_candidate_profiles_interstitial_acknowledged
+    before_action :check_ats_interstitial_acknowledged
 
     helper_method :current_user
 
     def check_terms_and_conditions
       redirect_to publishers_terms_and_conditions_path unless current_publisher.accepted_terms_at?
     end
 
-    def check_candidate_profiles_interstitial_acknowledged
-      redirect_to publishers_candidate_profiles_interstitial_path unless current_publisher.nil? || current_publisher.acknowledged_candidate_profiles_interstitial?
+    def check_ats_interstitial_acknowledged
+      return if current_publisher.nil? || current_publisher.acknowledged_ats_and_religious_form_interstitial?
+
+      redirect_to publishers_ats_interstitial_path
     end
   end
 end
diff --git a/app/controllers/publishers/candidate_profiles_interstitials_controller.rb b/app/controllers/publishers/candidate_profiles_interstitials_controller.rb
diff --git a/app/controllers/publishers/terms_and_conditions_controller.rb b/app/controllers/publishers/terms_and_conditions_controller.rb
@@ -1,6 +1,6 @@
 class Publishers::TermsAndConditionsController < Publishers::BaseController
   skip_before_action :check_terms_and_conditions, only: %i[show update]
-  skip_before_action :check_candidate_profiles_interstitial_acknowledged, only: %i[show update]
+  skip_before_action :check_ats_interstitial_acknowledged, only: %i[show update]
 
   def show
     @terms_and_conditions_form = Publishers::TermsAndConditionsForm.new
@@ -10,7 +10,7 @@ def update
     @terms_and_conditions_form = Publishers::TermsAndConditionsForm.new(terms_params)
     if @terms_and_conditions_form.valid?
       current_publisher.update(accepted_terms_at: Time.current)
-      redirect_to publishers_candidate_profiles_interstitial_path
+      redirect_to publishers_ats_interstitial_path
     else
       render :show
     end
diff --git a/app/models/school.rb b/app/models/school.rb
@@ -65,6 +65,10 @@ def faith_school?
     religious_character.present?
   end
 
+  def catholic_school?
+    religious_character&.include?("Catholic") || false
+  end
+
   def school_type
     read_attribute(:school_type).singularize
   end
@@ -90,4 +94,14 @@ def all_organisations
   def all_organisation_ids
     [id]
   end
+
+  def ats_interstitial_variant
+    if catholic_school?
+      "catholic"
+    elsif faith_school?
+      "other_faith"
+    else
+      "non_faith"
+    end
+  end
 end
diff --git a/app/models/school_group.rb b/app/models/school_group.rb
@@ -27,4 +27,8 @@ def all_organisations
   def all_organisation_ids
     [id] + schools.pluck(:id) + schools_outside_local_authority.pluck(:id)
   end
+
+  def ats_interstitial_variant
+    "non_faith"
+  end
 end
diff --git a/app/views/publishers/ats_interstitials/show.html.slim b/app/views/publishers/ats_interstitials/show.html.slim
@@ -0,0 +1,30 @@
+- scope = "ats_interstitials.#{@variant}"
+- content_for :page_title_prefix, t("ats_interstitials.shared.page_title")
+
+.govuk-grid-row
+  .govuk-grid-column-two-thirds
+    h1.govuk-heading-xl = t("ats_interstitials.shared.page_title")
+    p.govuk-body = t("ats_interstitials.shared.intro")
+
+    h2.govuk-heading-m = t("ats_interstitials.shared.ats_list_header")
+    ul.govuk-list.govuk-list--bullet
+      - t("ats_interstitials.shared.ats_points").each do |item|
+        li = item
+
+    - if I18n.exists?("#{scope}.faith_paragraph")
+      p.govuk-body = t("#{scope}.faith_paragraph")
+
+    - if I18n.exists?("#{scope}.faith_intro")
+      p.govuk-body = t("#{scope}.faith_intro")
+
+    - if I18n.exists?("#{scope}.faith_points")
+      ul.govuk-list.govuk-list--bullet
+        - t("#{scope}.faith_points").each do |item|
+          li = item
+
+    p.govuk-body
+      = govuk_link_to t("ats_interstitials.shared.guidance_link_text"),
+        post_path(section: "get-help-hiring", subcategory: "how-to-create-job-listings-and-accept-applications", post_name: "accepting-job-applications-on-teaching-vacancies")
+
+    .govuk-button-group
+      = govuk_button_to "Continue", publishers_ats_interstitial_path, method: :patch, id: "acknowledge-interstitial-button"
diff --git a/app/views/publishers/candidate_profiles_interstitials/show.html.slim b/app/views/publishers/candidate_profiles_interstitials/show.html.slim
diff --git a/config/analytics.yml b/config/analytics.yml
@@ -293,7 +293,7 @@ shared:
     - id
     - oid
     - accepted_terms_at
-    - acknowledged_candidate_profiles_interstitial
+    - acknowledged_ats_and_religious_form_interstitial
     - email
     - email_opt_out
     - last_activity_at
diff --git a/config/locales/en.yml b/config/locales/en.yml
@@ -400,17 +400,32 @@ en:
       show_count: Applications (%{count})
     statistics: Statistics
 
-  candidate_profiles_interstitials:
-    page_title: You can now view candidate profiles and invite them to apply to jobs
-    intro: A candidate's profile includes their job preferences, experience and qualifications.
-    list_header: "These candidates have specified:"
-    page_description_items:
-      - "that they're able to travel to your school"
-      - "that they're interested in an education phase that matches your school"
-    outro: You'll also be notified about any new candidate profiles that match your job listings.
-    button_text: View candidate profiles
-    skip_text: Skip to job listings
-
+  ats_interstitials:
+      shared:
+        page_title: "You can now use our Applicant Tracking System (ATS)"
+        intro: "We have launched our Applicant Tracking System (ATS). This means that when you use our Keeping Children Safe in Education (KCSIE) compliant application form, you can manage applications from the beginning to the end of the hiring process."
+        ats_list_header: "Using our ATS, you can:"
+        ats_points:
+          - "update multiple applications at once"
+          - "communicate with jobseekers"
+          - "keep track of interviews"
+          - "record notes on progress for colleagues"
+        guidance_link_text: "Read our How to accept job applications guidance to find out more."
+      catholic:
+        faith_paragraph: "You now also have the option to use our Catholic Education Service (CES) approved online application form."
+        faith_intro: "The form asks applicants about their:"
+        faith_points:
+          - "religion"
+          - "place of worship"
+          - "baptism"
+          - "religious referees"
+      other_faith:
+        faith_paragraph: "We offer a version of the form that asks applicants questions about religion. When you use this form, it gives applicants an opportunity to tell you:"
+        faith_points:
+          - "if they have a religious denomination or faith"
+          - "how they will meet your school’s ethos and aims"
+          - "their religious reference details"
+          - "their place of worship"
   candidate_profiles_banner:
     intro: 'Candidate profiles are a new feature that allows schools to contact you about relevant jobs. With a profile you can:'
     summary_list:
diff --git a/config/routes.rb b/config/routes.rb
@@ -261,7 +261,8 @@
       get "schools/preview", to: "/publishers/organisations/schools#preview"
     end
     resource :terms_and_conditions, only: %i[show update]
-    resource :candidate_profiles_interstitial, only: %i[show]
+    resource :ats_interstitial, only: %i[show update]
+    get :remove_organisation_filter, to: "publisher_preferences#remove_organisation"
   end
 
   scope :publishers do
diff --git a/db/migrate/20260112141400_add_ats_interstitial_to_publishers.rb b/db/migrate/20260112141400_add_ats_interstitial_to_publishers.rb
@@ -0,0 +1,8 @@
+class AddAtsInterstitialToPublishers < ActiveRecord::Migration[7.1]
+  def change
+    add_column :publishers, :acknowledged_ats_and_religious_form_interstitial, :boolean, default: false, null: false
+    safety_assured do
+      remove_column :publishers, :acknowledged_candidate_profiles_interstitial, :boolean, default: false
+    end
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
@@ -652,8 +652,8 @@
     t.text "given_name_ciphertext"
     t.datetime "dismissed_new_features_page_at", precision: nil
     t.datetime "unsubscribed_from_expired_vacancy_prompt_at", precision: nil
-    t.boolean "acknowledged_candidate_profiles_interstitial", default: false, null: false
     t.boolean "email_opt_out", default: false, null: false
+    t.boolean "acknowledged_ats_and_religious_form_interstitial", default: false, null: false
     t.index ["email"], name: "index_publishers_on_email"
     t.index ["oid"], name: "index_publishers_on_oid", unique: true
   end
diff --git a/spec/factories/publishers.rb b/spec/factories/publishers.rb
@@ -1,7 +1,7 @@
 FactoryBot.define do
   factory :publisher do
     accepted_terms_at { Date.current - 5.months }
-    acknowledged_candidate_profiles_interstitial { true }
+    acknowledged_ats_and_religious_form_interstitial { true }
     dismissed_new_features_page_at { Time.now }
     email { Faker::Internet.email(domain: "contoso.com") }
     family_name { Faker::Name.last_name.delete("'") }
diff --git a/spec/models/school_spec.rb b/spec/models/school_spec.rb
@@ -39,6 +39,70 @@
     end
   end
 
+  describe "#catholic_school?" do
+    subject { build(:school, gias_data: gias_data) }
+
+    context "when the religious character is Catholic" do
+      let(:gias_data) { { "ReligiousCharacter (name)" => "Catholic" } }
+
+      it "returns true" do
+        expect(subject.catholic_school?).to be true
+      end
+    end
+
+    context "when the religious character is Roman Catholic" do
+      let(:gias_data) { { "ReligiousCharacter (name)" => "Roman Catholic" } }
+
+      it "returns true" do
+        expect(subject.catholic_school?).to be true
+      end
+    end
+
+    context "when the school has no religious character" do
+      let(:gias_data) { { "ReligiousCharacter (name)" => "Does not apply" } }
+
+      it "returns false" do
+        expect(subject.catholic_school?).to be false
+      end
+    end
+
+    context "when the school has no gias_data" do
+      let(:gias_data) { nil }
+
+      it "returns false" do
+        expect(subject.catholic_school?).to be false
+      end
+    end
+  end
+
+  describe "#ats_interstitial_variant" do
+    subject { build(:school, gias_data: gias_data) }
+
+    context "when catholic_school? is true" do
+      let(:gias_data) { { "ReligiousCharacter (name)" => "Catholic" } }
+
+      it "returns catholic" do
+        expect(subject.ats_interstitial_variant).to eq("catholic")
+      end
+    end
+
+    context "when faith_school? is true but not catholic" do
+      let(:gias_data) { { "ReligiousCharacter (name)" => "Church of England" } }
+
+      it "returns other_faith" do
+        expect(subject.ats_interstitial_variant).to eq("other_faith")
+      end
+    end
+
+    context "when the school has no faith" do
+      let(:gias_data) { { "ReligiousCharacter (name)" => "Does not apply" } }
+
+      it "returns non_faith" do
+        expect(subject.ats_interstitial_variant).to eq("non_faith")
+      end
+    end
+  end
+
   describe "#urn" do
     it "must be unique" do
       create(:school, urn: "12345")
diff --git a/spec/requests/publishers/ats_interstitials_spec.rb b/spec/requests/publishers/ats_interstitials_spec.rb
@@ -0,0 +1,79 @@
+require "rails_helper"
+
+RSpec.describe "Publishers::AtsInterstitials" do
+  let(:publisher) { create(:publisher, acknowledged_ats_and_religious_form_interstitial: false) }
+  let(:organisation) { create(:school) }
+
+  before do
+    sign_in(publisher, scope: :publisher)
+    # rubocop:disable RSpec/AnyInstance
+    allow_any_instance_of(Publishers::BaseController).to receive(:current_organisation).and_return(organisation)
+    allow_any_instance_of(Publishers::BaseController).to receive(:current_publisher).and_return(publisher)
+    # rubocop:enable RSpec/AnyInstance
+  end
+
+  describe "GET #show" do
+    context "when the organisation is a SchoolGroup" do
+      let(:organisation) { create(:school_group) }
+
+      it "renders the show template successfully (hits :default branch)" do
+        get publishers_ats_interstitial_path
+        expect(response).to be_successful
+      end
+    end
+
+    context "when the organisation is not a SchoolGroup" do
+      let(:organisation) { create(:school) }
+
+      before do
+        allow(organisation).to receive(:ats_interstitial_variant).and_return(:custom_variant)
+      end
+
+      it "renders the show template successfully (hits 'else' branch)" do
+        get publishers_ats_interstitial_path
+        expect(response).to be_successful
+      end
+    end
+  end
+
+  describe "PATCH #update" do
+    context "when the update is successful" do
+      it "updates the publisher and redirects" do
+        expect {
+          patch publishers_ats_interstitial_path
+        }.to change { publisher.reload.acknowledged_ats_and_religious_form_interstitial }.from(false).to(true)
+
+        expect(response).to redirect_to(organisation_jobs_with_type_path)
+      end
+    end
+
+    context "when the update fails" do
+      before do
+        allow(publisher).to receive(:update).and_return(false)
+      end
+
+      it "re-renders the show template" do
+        patch publishers_ats_interstitial_path
+
+        expect(response).to render_template(:show)
+      end
+    end
+
+    context "when the update fails (SchoolGroup)" do
+      let(:organisation) { create(:school_group) }
+
+      before do
+        # rubocop:disable RSpec/AnyInstance
+        allow_any_instance_of(Publishers::BaseController).to receive(:current_organisation).and_return(organisation)
+        allow_any_instance_of(Publisher).to receive(:update).and_return(false)
+        # rubocop:enable RSpec/AnyInstance
+      end
+
+      it "renders show with :default variant" do
+        patch publishers_ats_interstitial_path
+        expect(assigns(:variant)).to eq("non_faith")
+        expect(response).to render_template(:show)
+      end
+    end
+  end
+end
diff --git a/spec/system/publishers/publishers_can_see_ats_interstitial_spec.rb b/spec/system/publishers/publishers_can_see_ats_interstitial_spec.rb
diff --git a/spec/system/publishers/publishers_can_view_candidate_interstitial_spec.rb b/spec/system/publishers/publishers_can_view_candidate_interstitial_spec.rb
diff --git a/spec/system/publishers/publishers_have_to_accept_terms_spec.rb b/spec/system/publishers/publishers_have_to_accept_terms_spec.rb
