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

Update Flask React post to use CLI for setup #686

Merged
merged 7 commits into from
Apr 5, 2021
Merged

Conversation

mraible
Copy link
Contributor

@mraible mraible commented Apr 3, 2021

@mraible mraible added the qa-needed This blog post is ready for QA label Apr 3, 2021
@netlify
Copy link

netlify bot commented Apr 3, 2021

Deploy preview for okta-blog ready!

Built with commit ae5d1a3

https://deploy-preview-686--okta-blog.netlify.app

@mraible mraible added qa-complete QA has been done on this post and removed qa-needed This blog post is ready for QA labels Apr 5, 2021
@jpf jpf merged commit 253c0ac into main Apr 5, 2021
@jpf jpf deleted the flask-react-update branch April 5, 2021 19:16
async componentDidMount() {
const accessToken = await this.props.authService.getAccessToken()
componentDidMount() {
const accessToken = this.props.authState.accessToken;
Copy link
Contributor Author

@mraible mraible Apr 5, 2021

Choose a reason for hiding this comment

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

This should be this.props.authState.accessToken.accessToken. See oktadev/python-flask-react-crud-example@ce09dd9#diff-97cac7ad88dabf433a1eeb4a3fee7582d168a6bc1fdddfd4a9a8cdc0ece808e6R35.

I created #690 to fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
qa-complete QA has been done on this post
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants