Skip to content

Comments

feat: Show SOI after Guardian jobs T&Cs#3324

Open
AshCorr wants to merge 3 commits intomainfrom
ash/consents-after-agree
Open

feat: Show SOI after Guardian jobs T&Cs#3324
AshCorr wants to merge 3 commits intomainfrom
ash/consents-after-agree

Conversation

@AshCorr
Copy link
Member

@AshCorr AshCorr commented Feb 16, 2026

What does this change?

  • Fixes a bug where the Guardian Jobs T&C's page would not be shown to users that already had a Guardian Account and were signing into Guardian Jobs for the first time. I believe this was a regression introduced by combining the registration and sign in flows.
  • Add the SOI page after the Guardian Jobs T&Cs page with JUST the Guardian Jobs Newsletter option if the user already had a guardian account, but is signing into Guardian Jobs for the first time.
  • Moves Guardian Jobs T&Cs page to BEFORE the SOI flow for both sign in and registration. This made the implementation of the previous feature slightly easier and makes sign in and registration a consistent flow for Guardian Jobs.

Before (Registration)

image

Before (Sign in - BROKEN)

We're currently not showing the Guardian Jobs T&Cs page for existing Guardian users that are signing into Guardian Jobs for the first time. This means we're not capturing necessary data like their first and last name, nor are we tagging them as a Guardian Jobs user which has implications on retention.

image

After (Registration)

image

After (Sign in - FIXED)

image

@github-actions
Copy link

github-actions bot commented Feb 16, 2026

@AshCorr AshCorr added run_tests Run Cypress & Chromatic feature Departmental tracking: work on a new feature labels Feb 17, 2026
@AshCorr AshCorr force-pushed the ash/consents-after-agree branch from 3dcf81b to e283430 Compare February 17, 2026 15:20
@AshCorr AshCorr added run_tests Run Cypress & Chromatic and removed run_tests Run Cypress & Chromatic labels Feb 17, 2026
@AshCorr AshCorr force-pushed the ash/consents-after-agree branch from e283430 to 57613c5 Compare February 18, 2026 08:07
@AshCorr AshCorr added run_tests Run Cypress & Chromatic and removed run_tests Run Cypress & Chromatic labels Feb 18, 2026
@AshCorr AshCorr force-pushed the ash/consents-after-agree branch from 57613c5 to 34601f3 Compare February 18, 2026 09:01
@AshCorr AshCorr added run_tests Run Cypress & Chromatic and removed run_tests Run Cypress & Chromatic labels Feb 18, 2026
@AshCorr AshCorr force-pushed the ash/consents-after-agree branch from 34601f3 to 9ce63c8 Compare February 18, 2026 09:32
@AshCorr AshCorr changed the title fix: Show Guardian Jobs T&Cs in combine sign in flow feat: Show SOI after Guardian jobs T&Cs Feb 18, 2026
@AshCorr AshCorr marked this pull request as ready for review February 18, 2026 09:33
@AshCorr AshCorr requested a review from a team as a code owner February 18, 2026 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Departmental tracking: work on a new feature run_tests Run Cypress & Chromatic

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant