Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Use OpenID Connect users with existing accounts #8178

Closed
wants to merge 2 commits into from

Conversation

OmmyZhang
Copy link
Contributor

fix #7633 and #8155

@clokep
Copy link
Member

clokep commented Aug 26, 2020

Can you merge develop into this for the conflicts / CI? Thank you!

@OmmyZhang
Copy link
Contributor Author

Fine, I need to resolve conflicts at first.

@OmmyZhang OmmyZhang closed this Aug 26, 2020
@clokep
Copy link
Member

clokep commented Aug 26, 2020

No reason to close this, you can push back to the same branch!

@OmmyZhang
Copy link
Contributor Author

OmmyZhang commented Aug 26, 2020

@clokep It seems a little complex, as I need to open a PR in BBBSnownball's repository to update this branch.
I have forked BBBSnownball's repository and I will open a new PR soon with my own branch.

@BBBSnowball
Copy link
Contributor

@OmmyZhang I have added you as a collaborator. Please feel free to use my repository or yours - whatever is easier for you.

Thanks for continuing this work :-)

@OmmyZhang
Copy link
Contributor Author

@BBBSnowball Thanks for your help! This work seems almost done. (pull #8180 )

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use OpenID Connect users with existing accounts
4 participants