Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating cookies policy #7181

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Updating cookies policy #7181

wants to merge 8 commits into from

Conversation

EllieNodder
Copy link
Collaborator

Trello card URL

https://trello.com/c/9tEzwEY3/1332-update-cookies-policy

Changes in this PR:

We need to update our cookies to account for VWO and Microsoft Clarity.

Screenshots of UI changes:

Before

After

Next steps:

  • Terraform deployment required?

  • New development configuration to be shared?

Copy link

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

EllieNodder and others added 5 commits October 23, 2024 16:47
* origin/main: (23 commits)
  Rubocop govuk (#7199)
  Only validate models email if email changed
  Improve application form content (#7167)
  Do not validate Org when creating a Vacancy
  Bump rtCamp/action-slack-notify from 2.3.0 to 2.3.2
  Bump the npm-dependencies group with 4 updates (#7196)
  Fixing typo and content tweaks on request an account page
  Adding optional to flexible working question
  Add tag to deadline passed on saved jobs
  Add blank line at end of file for linting
  Add missing button back
  Fix link on preview section
  Fix link
  fix syntax errors
  Add link to turn on profile text
  tweaking copy
  trying to fix link
  Tweaking turning profile on text and adding link
  Bump Openssl version in docker image
  Bump the gem-dependencies group with 5 updates
  ...
Unrelated to this PR. This should be failing in "main" branch!
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.

3 participants