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

AP-427 add met.police.uk email domain to the onboarding-self-service-experience allow list #960

Merged
merged 3 commits into from
Dec 13, 2024

Conversation

pauldougan
Copy link
Contributor

@pauldougan pauldougan commented Dec 3, 2024

AP-427

Onboarding Feature Deployment

Warning

Pull requests merged to main will be released to production, please ensure the checklist below is complete

Before any work can be merged to main in must meet the definition of done and be ready to deploy. While many of these tasks will be automated, the reviewers must take the responsibility of confirming the checklist below has been completed before this ticket can be merged.

Checklist

  • this pull request meets the acceptance criteria of the ticket

  • this branch is up-to-date with the main branch

    git fetch --all && git rebase origin/main

  • these changes are backwards compatible (no breaking changes)

    • all methods signatures and return values are the same
    • any replaced methods are marked as @deprecated
  • tests have been written to cover any new or updated functionality

  • new configuration parameters have been deployed to all environments, see configuration management.

  • all external infrastructure dependencies have been updated in all environments

Changes

[ please list the changes this pull request is making ]

Changed for changes in existing functionality

  • added met.police to allow list
  • removed redundant capitalised domains from the allow list
  • sort list of domains in alphabetical order

- remove duplicate capitalised domains
- sort domains into alphabetic sequence for readability
- insert met.police.uk domain
@pauldougan pauldougan requested a review from a team as a code owner December 3, 2024 15:20
@pauldougan pauldougan changed the title AP-427 - Projects Adoption Pod Backlog Add parent AP-427 add met.police.uk email domain to the onboarding-product-page and onboarding-self-service-experience allow lists AP-427 add met.police.uk email domain to the onboarding-product-page and onboarding-self-service-experience allow lists Dec 3, 2024
@pauldougan pauldougan changed the title AP-427 add met.police.uk email domain to the onboarding-product-page and onboarding-self-service-experience allow lists AP-427 add met.police.uk email domain to the onboarding-self-service-experience allow list Dec 3, 2024
@pauldougan pauldougan requested a review from a team December 3, 2024 15:24
CarlyG55
CarlyG55 previously approved these changes Dec 11, 2024
@pauldougan
Copy link
Contributor Author

Hi @ethanmills if you can review this it would be helpful and unblock the service team

@philf999 philf999 added this pull request to the merge queue Dec 13, 2024
Merged via the queue into main with commit ed830c5 Dec 13, 2024
15 of 16 checks passed
@philf999 philf999 deleted the AP-427 branch December 13, 2024 10:09
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.

4 participants