-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
website/docs: Update Azure AD identity provider doc page #14476
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
Conversation
Describe how to add Azure AD source to login screen. Signed-off-by: Jan Skála <skalahonza@users.noreply.github.com>
✅ Deploy Preview for authentik-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
All reactions
Sorry, something went wrong.
✅ Deploy Preview for authentik-storybook canceled.
|
All reactions
Sorry, something went wrong.
@@ -111,6 +111,23 @@ return True | |||
|
|||
Try to login with a **_new_** user. You should see no prompts and the user should have the correct information. | |||
|
|||
### Azure AD source on login screen |
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 goes for every social login source and I don't think we include this step-by-step on other pages.
Tho, if we decide to go ahead and include this:
- authentik should always be written in lowercase
- Please use a numbered list for this
- You can't assume flows/stages (e.g. default-authentication-flow) as users may use different flows or they were renamed
- You can't assume the name either as some might choose a non-standard name
Sorry, something went wrong.
All reactions
-
👍 1 reaction
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.
@dominic-r
I’ve updated the code to address your points
Sorry, something went wrong.
All reactions
Hi @skalahonza. Thank you for this suggestion and contribution. We're going to handle this slightly differently by referring to an existing guide, rather than adding the guide to each source document. I've created a PR for this: #14667 So I'm going to close this PR in favor of that different method. Regardless, thank you for pointing this out. |
All reactions
-
❤️ 1 reaction
Sorry, something went wrong.
dominic-r
Successfully merging this pull request may close these issues.
None yet
Details
Describe how to add Azure AD source to login screen.
Checklist
ak test authentik/
)make lint-fix
)If an API change has been made
make gen-build
)If changes to the frontend have been made
make web
)If applicable
make website
)