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.

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

Revert "Provision GitHub OAuth using secret (#1773)" #1880

Merged
merged 1 commit into from
Mar 3, 2021
Merged

Revert "Provision GitHub OAuth using secret (#1773)" #1880

merged 1 commit into from
Mar 3, 2021

Conversation

themr0c
Copy link
Contributor

@themr0c themr0c commented Mar 2, 2021

This reverts commit 478d1d1. That should not be in the 7.24.x branch.

@themr0c themr0c added this to the 7.24.x milestone Mar 2, 2021
@themr0c themr0c requested review from yhontyk, MichalMaler, rkratky and a team as code owners March 2, 2021 15:44
@themr0c themr0c requested review from benoitf and removed request for a team March 2, 2021 15:44
@themr0c themr0c marked this pull request as draft March 2, 2021 15:49
@themr0c
Copy link
Contributor Author

themr0c commented Mar 2, 2021

@dmytro-ndp @tolusha I put this PR as WIP as long as we haven't decided if this feature needs rollback or not.

@dmytro-ndp
Copy link
Contributor

dmytro-ndp commented Mar 2, 2021

It's a correct fix because describing feature is absent in Eclipse Che 7.24.x operator version https://github.com/eclipse/che-operator/commits/7.24.2:
Screenshot from 2021-03-02 19-27-56

@themr0c themr0c marked this pull request as ready for review March 3, 2021 12:54
@themr0c themr0c merged commit ee61f27 into eclipse-che:7.24.x Mar 3, 2021
@themr0c themr0c deleted the revert-github-oauth branch March 3, 2021 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants