-
Notifications
You must be signed in to change notification settings - Fork 43
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
Fixes #733 - Add M365 Graph channel guide #738
Conversation
Hi @dominikklein & @mantas, can you please have a look at the changes in the PR? @dvuckovic already had a look and suggested to also ask you for feedback. It would be interesting if you could especially have a look at the common errors section if there is something missing. |
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.
In general nice work :-) Only some small comments/questions.
Personally I would love some screenshots for some important things, at least to give some better feeling as an addition to the only text description (e.g. finding correct places in entry, but maybe the people are normaly experts in Microsoft world?).
I saw that nothing related to admin consent is mentioned and also nothing about re-authentication.
I added 2 small sections at the bottom of the page about it.
I added now some screenshots. Let me know if you consider them the right ones ;-) |
No description provided.