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

ER-876 gov one terms and conditions page #980

Merged
merged 8 commits into from
Dec 5, 2023

Conversation

jack-coggin
Copy link
Contributor

@jack-coggin jack-coggin commented Nov 27, 2023

ER-876

Create terms and conditions page for Gov One Login users creating an account. This will be the first step of the registration flow

@jack-coggin jack-coggin marked this pull request as ready for review November 27, 2023 17:24
@jack-coggin jack-coggin added the review Review app deployed to Azure label Nov 27, 2023
@jack-coggin jack-coggin changed the title Er 876 gov one t and c page Er 876 gov one terms and conditions page Nov 27, 2023
@jack-coggin jack-coggin changed the title Er 876 gov one terms and conditions page ER-876 gov one terms and conditions page Nov 27, 2023
Copy link

@instantrick
Copy link

All looks good to me @jack-coggin

@peterdavidhamilton
Copy link
Contributor

@jack-coggin can this PR be merged independently or is it dependent upon a working GovOne authentication mechanism?

@peterdavidhamilton peterdavidhamilton added this to the rc0.13.0 milestone Dec 1, 2023
@jack-coggin
Copy link
Contributor Author

@jack-coggin can this PR be merged independently or is it dependent upon a working GovOne authentication mechanism?

I think the only reference to Gov One that's in here is an update to the omniauth_callbacks_controller redirect

@jack-coggin jack-coggin merged commit 879fbed into one-login Dec 5, 2023
3 of 4 checks passed
@peterdavidhamilton peterdavidhamilton deleted the ER-876-gov-one-t-and-c-page branch January 8, 2024 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
review Review app deployed to Azure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants