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

Update privacy link #991

Merged
merged 2 commits into from
Feb 14, 2024
Merged

Update privacy link #991

merged 2 commits into from
Feb 14, 2024

Conversation

gpeng
Copy link
Contributor

@gpeng gpeng commented Jan 18, 2024

Currently DNM as we're unsure the new privacy policy covers everything for this service.

Context

We're now using a DfE wide privacy policy.

Changes proposed in this pull request

Update accordingly.

Guidance to review

Checklist

  • Attach to Trello card
  • Rebased main
  • Cleaned commit history
  • Tested by running locally

@gpeng gpeng requested a review from a team as a code owner January 18, 2024 16:28
@gpeng
Copy link
Contributor Author

gpeng commented Jan 18, 2024

@gpeng gpeng changed the title Update privacy link [DO NOT MERGE] Update privacy link Jan 19, 2024
gpeng added 2 commits February 7, 2024 10:13
We're now using a DfE wide privacy policy.

Update accordingly.
* Redirect /privacy route to the DfE wide policy
* Remove the existing views and controller action
@gpeng gpeng force-pushed the update-privacy-link branch from 088eb1d to 086487c Compare February 7, 2024 10:13
@gpeng gpeng changed the title [DO NOT MERGE] Update privacy link Update privacy link Feb 7, 2024
@gpeng gpeng merged commit 5d79d19 into main Feb 14, 2024
11 checks passed
@gpeng gpeng deleted the update-privacy-link branch February 14, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants