docs: Add Microsoft Identity Provider#418
docs: Add Microsoft Identity Provider#418marcelomendoncasoares merged 3 commits intoserverpod:mainfrom
Conversation
8ead32e to
1f42261
Compare
marcelomendoncasoares
left a comment
There was a problem hiding this comment.
Great work with the docs, @vfiruz97! A few suggestions for improvement.
docs/06-concepts/11-authentication/04-providers/07-microsoft/01-setup.md
Outdated
Show resolved
Hide resolved
docs/06-concepts/11-authentication/04-providers/07-microsoft/01-setup.md
Outdated
Show resolved
Hide resolved
docs/06-concepts/11-authentication/04-providers/07-microsoft/01-setup.md
Outdated
Show resolved
Hide resolved
docs/06-concepts/11-authentication/04-providers/07-microsoft/01-setup.md
Outdated
Show resolved
Hide resolved
docs/06-concepts/11-authentication/04-providers/07-microsoft/02-configuration.md
Show resolved
Hide resolved
docs/06-concepts/11-authentication/04-providers/07-microsoft/02-configuration.md
Show resolved
Hide resolved
|
Thanks @marcelomendoncasoares, for a quick feedback. The comments are addressed. Ready for another review. |
|
Thanks @marcelomendoncasoares, appreciate your efforts. My suggestion, first merge this PR and then reorder in other PR. |
Agree with the suggestion to move Facebook up to position 4. We should then also make the anonymous 1 (instead of 0 as it is now) in the same PR. |
Hi @marcelomendoncasoares,
This PR adds a doc for serverpod/serverpod#4716. Please review when you have a chance.