Skip to content

Conversation

@skabou
Copy link
Contributor

@skabou skabou commented Jan 14, 2022

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.

Copy link
Member

@ckittel ckittel left a 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>
Copy link
Member

@ckittel ckittel left a 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.

skabou and others added 5 commits January 18, 2022 14:13
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 ckittel self-requested a review January 18, 2022 22:39
Copy link
Member

@ckittel ckittel left a 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!

Copy link
Contributor

@mmacy mmacy left a 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.


## 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.
Copy link
Contributor

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.

Copy link
Contributor Author

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

skabou and others added 2 commits January 20, 2022 14:44
Co-authored-by: Marsh Macy <mmacy@users.noreply.github.com>
Updated "About the code"
Copy link
Contributor

@mmacy mmacy left a comment

Choose a reason for hiding this comment

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

LGTM! :shipit:

@skabou skabou merged commit 3db9bb2 into Azure-Samples:main Jan 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants