-
Notifications
You must be signed in to change notification settings - Fork 984
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
Adding entra ID sso saml migration #6779
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -113,3 +113,32 @@ You must complete the domain authorization before you toggle `Enable New SSO Aut | |||
::: | |||
|
|||
<Lightbox src="/img/docs/dbt-cloud/access-control/azure-enable.png" title="Enable new SSO"/> | |||
|
|||
## Google Workspace |
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.
This section is not new, just moved
|
||
::: | ||
<Lightbox src="/img/docs/dbt-cloud/access-control/saml-enable-entra.png" title="Enable new SSO for SAML/Okta"/> |
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.
{slug}
placeholder here instead of entra-id
looks good, just needs the changes Ajay called out and then I'll approve |
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.
Just a small suggestion about placeholder text consistency
|
||
<Lightbox src="/img/docs/dbt-cloud/access-control/google-cloud-sso.png" title="Google Cloud Console"/> | ||
The Entra ID fields that will be updated are: | ||
- Single sign-on URL — `https://<YOUR_AUTH0_URI>/login/callback?connection={slug}` |
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 realize this is copying what's above, but it might be worth it to use <SLUG>
instead of {slug}
to be consistent with our other placeholder text. Of course, you'd then need to adjust the screenshot
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.
@matthewshaver added 2 change requests, and then we're good. Thanks!
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.
Left a comment on one of the screenshots. Can merge after that fix.
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.
@matthewshaver I see the slug is still docs-account
for the Identifier (Entity ID) field
What are you changing in this pull request and why?
Adds a new SAML 2.0 section for Entra ID in the migration guide - it moves the previous Entra ID up and Google Workspace down but these pages haven't been altered
Adds some clarifying language to the original Entra ID setup guide
Checklist
🚀 Deployment available! Here are the direct links to the updated files: