Skip to content

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

Closed
wants to merge 4 commits into from

Conversation

skalahonza
Copy link

@skalahonza skalahonza commented May 12, 2025

Details

Describe how to add Azure AD source to login screen.

Checklist

  • Local tests pass (ak test authentik/)
  • The code has been formatted (make lint-fix)

If an API change has been made

  • The API schema has been updated (make gen-build)

If changes to the frontend have been made

  • The code has been formatted (make web)

If applicable

  • The documentation has been updated
  • The documentation has been formatted (make website)
Page not found · GitHub · GitHub
Skip to content
404 “This is not the web page you are looking for”
Describe how to add Azure AD source to login screen.

Signed-off-by: Jan Skála <skalahonza@users.noreply.github.com>
@skalahonza skalahonza requested a review from a team as a code owner May 12, 2025 07:37
Copy link

netlify bot commented May 12, 2025

Deploy Preview for authentik-docs ready!

Name Link
🔨 Latest commit bb6d1db
🔍 Latest deploy log https://app.netlify.com/projects/authentik-docs/deploys/68302c4f6efe3c0008d35d41
😎 Deploy Preview https://deploy-preview-14476--authentik-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link

netlify bot commented May 12, 2025

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit bb6d1db
🔍 Latest deploy log https://app.netlify.com/projects/authentik-storybook/deploys/68302c4f5026df00089f9cc8

@skalahonza skalahonza changed the title Update Azure AD identity provider doc page website/docs: Update Azure AD identity provider doc page May 12, 2025
@@ -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
Copy link
Contributor

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

Copy link
Author

@skalahonza skalahonza May 23, 2025

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

@skalahonza skalahonza requested a review from dominic-r May 23, 2025 08:06
@dewi-tik
Copy link
Contributor

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.

@dewi-tik dewi-tik closed this May 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants