-
Notifications
You must be signed in to change notification settings - Fork 3
get token from callback_context #41
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
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 looks good to me 💃
I changed to have the token from the request cookies first since this is also called from other routes. Also moved the flask context check to allow the function to be called from celery. Fix https://github.com/plotly/dekn/issues/8953 @michaelbabyn Can you have a look again? Thank you. |
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 works after the change to dash that I mentioned in plotly/dash#3051 💃
@T4rk1n is this good to merge? |
Yes, just need to test it works with the latest dash version. |
Tested and working on de5 |
Take the token from the callback_context added in plotly/dash#3051
Fix
get_username
called inside a background callback.