Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

google: Document for service account-less access #74

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

whs
Copy link

@whs whs commented Dec 28, 2020

Document for dexidp/dex#1896

@nate-double-u
Copy link
Contributor

Hi @whs, thanks for updating the docs!

Please note, this repo requires commits to be DCO signed.

@nate-double-u
Copy link
Contributor

@whs I'm not able to do a technical review on this project, but I did a copy edit pass.

whs and others added 2 commits December 29, 2020 14:32
Signed-off-by: Manatsawin Hanmongkolchai <git@whs.in.th>
Co-authored-by: Nate W. <4453979+nate-double-u@users.noreply.github.com>
Signed-off-by: Manatsawin Hanmongkolchai <git@whs.in.th>
@whs whs force-pushed the whs/sa-less-group branch from 6dfecca to 8c0c550 Compare December 29, 2020 07:32
@whs
Copy link
Author

whs commented Dec 29, 2020

Thanks for the review, I applied the suggestion and added DCO. Now we wait for the code PR to merge.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants