Skip to content
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

Improve Azure AD Docs (Lombiq Technologies: OCORE-112) #13063

Merged
merged 5 commits into from
Jan 13, 2023

Conversation

Piedone
Copy link
Member

@Piedone Piedone commented Jan 9, 2023

I found a few errors and ommissions, so after figuring out what needs to be configured to make login work, I thought I'd write it down too.

- TenantId: Provide the Directory ID value from the Azure Active Directory properties
- CallbackPath: The request path within the application's base path where the user-agent will be returned. The middleware will process this request when it arrives.
If no value is provided, setup Azure AD app to use the default path /signin-oidc.
- Display Name: The text that'll be displayed on the Orchard login screen. We recommend something like "My Company Microsoft account".
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we move these in a tabular format if it's possible

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It certainly is, but I don't see how that would be nicer here (and it would be quite ugly in Markdown).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's there in almost if not all the settings. Not an issue

Co-authored-by: Hisham Bin Ateya <[email protected]>
@Piedone Piedone requested a review from hishamco January 9, 2023 23:52
@Piedone Piedone merged commit 4539be8 into OrchardCMS:main Jan 13, 2023
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.

2 participants