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
Move some OAuth of out of accounts-base #12202
Conversation
Fix tests and move few things back.
We are making some changes in the accounts(adding async) I think it makes sense to move this change to 2.9. but I'm all ears if you want this could go in a patch for 2.8.x |
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
Let's merge this into 2.9. |
# Conflicts: # docs/history.md
@denihs this will require a bit of a testing during the beta release phase to make sure that I haven't missed some use case and that with all the other changes going on around that nothing has been missed or overridden. |
Following the discussion in #12171 I have taken the initiative and moved some OAuth related code out of
accounts-base
intoaccounts-oauth
.I have also added some additional conditionals in 6196453, which can be cherry picked if needed, which might have caused the error seen in #12171.