- Notifications
You must be signed in to change notification settings - Fork 31
Initial import of "Device Code flow in Python console app" #5
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
ckittel left a comment
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.
Nice, here's my eng review feedback. Nice work.
Co-authored-by: Chad Kittel <chad.kittel@gmail.com>
ckittel left a comment
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 is in addition to the prior review, with only has one open comment.
Updated with formatting from 'black'
Co-authored-by: Chad Kittel <chad.kittel@gmail.com>
"web APP" -> "console"
Co-authored-by: Chad Kittel <chad.kittel@gmail.com>
ckittel left a comment
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.
Looks great from a C&L Engineering review perspective. Thanks!
mmacy left a comment
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.
Thanks, @skabou, some required changes on this one.
msal-device-flow-python/README.md Outdated
| | ||
| ## About the code | ||
| | ||
| This Python application will request a user to authenticate via the web browser. Upon successful authentication a request to the Graph API will be made and the response will be displayed. |
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 currently describes what the application does rather than the code that enables it.
The intent of this section is to provide a more technical (yet still high-level) overview of how the code itself works. Things like which auth library (or libraries) it uses, the primary library types it uses, how the redirects and auth flow goes... that sort of thing.
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.
Thanks - I'll work on an update
Co-authored-by: Marsh Macy <mmacy@users.noreply.github.com>
Updated "About the code"
mmacy left a comment
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.
LGTM! ![]()
This is for #DOCS-CODE-032.
Simple web app written in Python 3 that demonstrates how to authenticate a user to call Graph via the Device Code flow.