-
Notifications
You must be signed in to change notification settings - Fork 182
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
Clarify steps for adding an OIDC provider #4150
Conversation
👋 🤖 ✅ Looks like the changes were ported across versions, nice job! 🎉 You can read more about the versioning within our docs in our documentation guidelines. |
5adc870
to
ece11f4
Compare
ece11f4
to
8e68036
Compare
I deployed this specifically so reviewers could check out the markdown > html update in action in the TOC: Compare to live, where the TOC includes apparently duplicated links, and only the ones in the active tab work, which is a disruptive experience: https://docs.camunda.io/docs/next/self-managed/setup/guides/connect-to-an-oidc-provider/ |
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 one comment but its NABD for me :)
|
||
4. Set the following environment variables for the component you are configuring an app for: | ||
1. Within the Entra ID admin center, [register a new application](https://learn.microsoft.com/en-us/entra/identity-platform/quickstart-register-app) for **each** component you would like to connect. |
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 wondering, if we have this in here for **each** component you would like to connect
do we need the note above? Maybe though this is a case of making this point very clear :)
🧹 Preview environment for this PR has been torn down. |
Description
Closes #3952
The OIDC configuration instructions for MS Entra were misleading, and implied that you needed one application for every component.
When should this change go live?
hold
label or convert to draft PR)PR Checklist
/versioned_docs
directory./docs
directory (aka/next/
).