Skip to content

Add Ats notice on sign in#8330

Open
richardpattinson wants to merge 1 commit intomainfrom
2367-hiring-staff-notice-upon-sign-in
Open

Add Ats notice on sign in#8330
richardpattinson wants to merge 1 commit intomainfrom
2367-hiring-staff-notice-upon-sign-in

Conversation

@richardpattinson
Copy link
Contributor

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.)
If any of the above options has changed then the author must check/resolve all of the following...

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

@richardpattinson richardpattinson force-pushed the 2367-hiring-staff-notice-upon-sign-in branch 3 times, most recently from 83bf5fc to 3b2ae19 Compare November 26, 2025 11:18
@richardpattinson richardpattinson force-pushed the 2367-hiring-staff-notice-upon-sign-in branch 3 times, most recently from 208686b to 8bc4a65 Compare December 5, 2025 09:40
Copy link
Collaborator

@scruti scruti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think some key bits are missing to turn this into an "acknowledge once" interstitial flow as the one it is replacing.

Also left a few suggestions around!

@richardpattinson richardpattinson force-pushed the 2367-hiring-staff-notice-upon-sign-in branch 15 times, most recently from 4ab8f15 to 3e0d380 Compare January 16, 2026 16:14
@richardpattinson richardpattinson force-pushed the 2367-hiring-staff-notice-upon-sign-in branch 3 times, most recently from e50ba0a to 6327b6f Compare January 26, 2026 16:06
@richardpattinson richardpattinson force-pushed the 2367-hiring-staff-notice-upon-sign-in branch from 6327b6f to 35858c4 Compare January 26, 2026 16:22
@github-actions
Copy link

Review app deployed to https://teaching-vacancies-review-pr-8330.test.teacherservices.cloud on AKS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants