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.

Sign up for GitHub

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

Jump to bottom

feat: adds new external account authorized user credentials #1160

Merged
merged 21 commits into from Oct 14, 2022
Merged

feat: adds new external account authorized user credentials #1160

merged 21 commits into from Oct 14, 2022
+948 −27

Conversation

ScruffyProdigy
Copy link
Contributor

No description provided.

@ScruffyProdigy ScruffyProdigy added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 7, 2022
@lsirac lsirac changed the title feat: implement headful flow for external accounts feat: adds new external account authorized user credentials Oct 7, 2022
sai-sunder-s
sai-sunder-s requested changes Oct 10, 2022
View reviewed changes
google/auth/_default.py Outdated Show resolved Hide resolved
google/auth/headful.py Outdated Show resolved Hide resolved
google/oauth2/sts.py Outdated Show resolved Hide resolved
google/oauth2/sts.py Show resolved Hide resolved
google/auth/headful.py Outdated Show resolved Hide resolved
lsirac
lsirac reviewed Oct 10, 2022
View reviewed changes
google/auth/headful.py Outdated Show resolved Hide resolved
google/auth/headful.py Outdated Show resolved Hide resolved
google/auth/headful.py Outdated Show resolved Hide resolved
google/auth/headful.py Outdated Show resolved Hide resolved
google/auth/headful.py Outdated Show resolved Hide resolved
google/auth/_default.py Outdated Show resolved Hide resolved
google/auth/_default.py Show resolved Hide resolved
lsirac
lsirac reviewed Oct 12, 2022
View reviewed changes
google/auth/external_account.py Outdated Show resolved Hide resolved
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
google/auth/_default.py Outdated Show resolved Hide resolved
@ScruffyProdigy ScruffyProdigy removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 12, 2022
lsirac
lsirac reviewed Oct 12, 2022
View reviewed changes
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
google/auth/external_account_authorized_user.py Show resolved Hide resolved
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
tests/test_external_account_authorized_user.py Outdated Show resolved Hide resolved
tests/test_external_account_authorized_user.py Outdated Show resolved Hide resolved
tests/test_external_account_authorized_user.py Show resolved Hide resolved
tests/test_external_account_authorized_user.py Outdated Show resolved Hide resolved
sai-sunder-s
sai-sunder-s reviewed Oct 12, 2022
View reviewed changes
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
tests/test__default.py Outdated Show resolved Hide resolved
lsirac
lsirac reviewed Oct 13, 2022
View reviewed changes
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
google/auth/external_account_authorized_user.py Show resolved Hide resolved
google/auth/external_account_authorized_user.py Show resolved Hide resolved
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
google/auth/external_account_authorized_user.py Show resolved Hide resolved
lsirac
lsirac approved these changes Oct 13, 2022
View reviewed changes
google/auth/external_account_authorized_user.py Outdated Show resolved Hide resolved
@ScruffyProdigy ScruffyProdigy added the kokoro:run Add this label to force Kokoro to re-run the tests. label Oct 14, 2022
@yoshi-kokoro yoshi-kokoro removed the kokoro:run Add this label to force Kokoro to re-run the tests. label Oct 14, 2022
@ScruffyProdigy ScruffyProdigy added kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Oct 14, 2022
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Oct 14, 2022
@sai-sunder-s sai-sunder-s added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 14, 2022
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 14, 2022
@ScruffyProdigy ScruffyProdigy added the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 14, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 14, 2022
@ScruffyProdigy ScruffyProdigy merged commit 523f811 into googleapis:main Oct 14, 2022
@ScruffyProdigy ScruffyProdigy deleted the headful branch October 14, 2022 04:52
@release-please release-please bot mentioned this pull request Oct 14, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sai-sunder-s sai-sunder-s sai-sunder-s approved these changes

@lsirac lsirac lsirac approved these changes

@arithmetic1728 arithmetic1728 Awaiting requested review from arithmetic1728

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@ScruffyProdigy @sai-sunder-s @lsirac @yoshi-kokoro