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

Restore the authorisation opt-out functionality #1098

Merged
merged 2 commits into from
Apr 22, 2024
Merged

Restore the authorisation opt-out functionality #1098

merged 2 commits into from
Apr 22, 2024

Conversation

vinokurig
Copy link
Contributor

@vinokurig vinokurig commented Apr 19, 2024

What does this PR do?

What issues does this PR fix or reference?

https://issues.redhat.com/browse/CRW-6245

Is it tested? How?

  1. Configure GitHub oauth
  2. Start a workspace from a GitHub repository with devfile.
  3. When the authorisation page appears, click the Cancel button.
  4. Go back to Dashboard -> User Preferences -> Git Services tab
  5. See: a yellow warning sign is present in the GitHub item, click the kebab icon and then the Clear menu button.
  6. See: the yellow sign is disappeared.

Release Notes

Docs PR

@che-bot
Copy link
Contributor

che-bot commented Apr 19, 2024

Click here to review and test in web IDE: Contribute

Signed-off-by: ivinokur <ivinokur@redhat.com>
Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1098

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1098", name: che-dashboard}]}}]"

1 similar comment
Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1098

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1098", name: che-dashboard}]}}]"

@vinokurig vinokurig force-pushed the CRW-6245 branch 2 times, most recently from dfc5414 to 15cb84c Compare April 19, 2024 12:55
Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1098

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1098", name: che-dashboard}]}}]"

1 similar comment
Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1098

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1098", name: che-dashboard}]}}]"

@ibuziuk ibuziuk requested a review from artaleks9 April 19, 2024 16:07
@ibuziuk
Copy link
Member

ibuziuk commented Apr 19, 2024

/retest

Copy link

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-1098

kubectl patch command
kubectl patch -n eclipse-che "checluster/eclipse-che" --type=json -p="[{"op": "replace", "path": "/spec/components/dashboard/deployment", "value": {containers: [{image: "quay.io/eclipse/che-dashboard:pr-1098", name: che-dashboard}]}}]"

@artaleks9
Copy link
Contributor

Verified on Eclipse Che with quay.io/eclipse/che-dashboard:pr-1098:

  • GitHub.com - checking according to PR fix description - works as expected
  • Bitbucket Server OAuth2 - checking according to Jira issue - works properly

Copy link

openshift-ci bot commented Apr 22, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: akurinnoy, artaleks9, vinokurig

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@vinokurig vinokurig merged commit ad970c4 into main Apr 22, 2024
16 of 17 checks passed
@vinokurig vinokurig deleted the CRW-6245 branch April 22, 2024 12:29
@devstudio-release
Copy link

Build 3.14 :: dashboard_3.x/482: Console, Changes, Git Data

@devstudio-release
Copy link

@devstudio-release
Copy link

vinokurig added a commit that referenced this pull request Apr 22, 2024
Always decode url on factory url accept.
Restore the skip oauth functionality.
@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.14 :: dashboard_3.x/482: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/6581 triggered

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.14 :: get-sources-rhpkg-container-build_3.x/6538: FAILURE

devspaces-operator-bundle : 3.x :: Failed in 60682577 : BREW:BUILD/STATUS:UNKNOWN
FAILURE:; copied to quay

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

Successfully merging this pull request may close these issues.

6 participants