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

✨ Improve GitHub Access Flow for Justice Users #80

Merged
merged 13 commits into from
Jan 16, 2024

Conversation

jasonBirchall
Copy link
Member

@jasonBirchall jasonBirchall commented Jan 16, 2024

👀 Purpose

♻️ What's changed

  • ✨ Extend Conditional Logic for Justice Emails: Enhanced the conditional statements in our logic to include handling for @justice.gov.uk email addresses, ensuring their access flow is correctly managed.

  • 🎨 User Flow Error Handling: Improved error messages and handling mechanisms throughout the user flow, ensuring users are adequately informed in case of issues or discrepancies.

  • ✨ Instructions for Justice.gov.uk Users: Added clear, user-friendly instructions on the new justice-user.html template for @justice.gov.uk users, guiding them through the Azure AD authentication process.

📝 Notes

  • This is what the page will look like if it's merged:
image

Copy link
Contributor

github-actions bot commented Jan 16, 2024

🦙 MegaLinter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Elapsed time
❌ HTML htmlhint 2 2 0.31s
✅ PYTHON black 2 1 0 0.77s
✅ PYTHON flake8 2 0 0.46s
✅ PYTHON isort 2 2 0 0.2s
❌ PYTHON pylint 2 2 2.66s
✅ REPOSITORY gitleaks yes no 0.35s
❌ REPOSITORY trivy yes 1 5.18s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@jasonBirchall jasonBirchall changed the title ✨ Add journey for justice.gov.uk users ✨ Improve GitHub Access Flow for Justice Users Jan 16, 2024
PepperMoJ
PepperMoJ previously approved these changes Jan 16, 2024
@jasonBirchall jasonBirchall merged commit a0a0c7c into main Jan 16, 2024
3 of 4 checks passed
@jasonBirchall jasonBirchall deleted the conditional-orgs branch January 16, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants