-
Notifications
You must be signed in to change notification settings - Fork 15
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
Use Entra ID throughout #1869
Use Entra ID throughout #1869
Conversation
@jemrobinson I expect it should just be
https://learn.microsoft.com/en-us/entra/fundamentals/how-to-rename-azure-ad?source=recommendations See also https://learn.microsoft.com/en-us/entra/fundamentals/how-to-find-tenant |
Thanks @craddm! Do you see any other names/variable names that look wrong? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've highlighted quite a few here - the general rule is if something is a feature, it is just Microsoft Entra <x>
rather than Microsoft Entra ID <x>
, where if it is the whole product i.e. Azure Active Directory
it is Microsoft Entra ID
60eaa6e
to
e825812
Compare
data_safe_haven/infrastructure/components/dynamic/entra_id_application.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Matt Craddock <[email protected]>
f5ffc41
to
0e8809d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
af9ebf5
to
3fba250
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks both! This was a really irritating one to write and to review. |
✅ Checklist
Enable foobar integration
rather than515 foobar
).develop
.'[WIP]'
to the title if needed (if you're not yet ready to merge).Change from using AzureAD to Entra ID throughout
🌂 Related issues
Closes #1664
🔬 Tests
Not tested