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

Add a way to reject the authorisation opt-out flag from the dashboard #22600

Closed
vinokurig opened this issue Oct 12, 2023 · 1 comment · Fixed by eclipse-che/che-dashboard#972
Closed

Add a way to reject the authorisation opt-out flag from the dashboard #22600

vinokurig opened this issue Oct 12, 2023 · 1 comment · Fixed by eclipse-che/che-dashboard#972
Assignees
@olexii4
Labels
area/dashboard kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P1 Has a major impact to usage or development of the system. team/A This team is responsible for the Che Operator and all its operands as well as chectl and Hosted Che

Comments

@vinokurig
Copy link
Contributor

Is your task related to a problem? Please describe

when eclipse-che/che-server#576 is merged, the only way to reject the authorisation opt-out flag would be manually editing the workspace-preferences config-map.

Describe the solution you'd like

Update the Git Services tab of the User Preferences dashboard page:

  • When an SCM authorisation is rejected add the corresponding item to the Git Services tab.
  • The tab should have some additional label to mark it as an authorisation reject item.
  • When the workspace create/start page is reloaded after the authorisation reject (url with the error_code=access_denied query parameter) show a notification with the description that the authorisation page will not appear for the specific SCM provider, and some instructions how to revert the option back from the Git Services tab.

Describe alternatives you've considered

No response

Additional context

No response
@vinokurig vinokurig added kind/task Internal things, technical debt, and to-do tasks to be performed. area/dashboard team/A This team is responsible for the Che Operator and all its operands as well as chectl and Hosted Che labels Oct 12, 2023
@che-bot che-bot added the status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. label Oct 12, 2023
@vinokurig vinokurig added kind/task Internal things, technical debt, and to-do tasks to be performed. and removed kind/task Internal things, technical debt, and to-do tasks to be performed. labels Oct 12, 2023
@vinokurig vinokurig mentioned this issue Oct 12, 2023
Merged
9 tasks
@amisevsk amisevsk added severity/P2 Has a minor but important impact to the usage or development of the system. severity/P1 Has a major impact to usage or development of the system. and removed status/need-triage An issue that needs to be prioritized by the curator responsible for the triage. See https://github. severity/P2 Has a minor but important impact to the usage or development of the system. labels Oct 13, 2023
@akurinnoy
Copy link
Contributor

akurinnoy commented Oct 25, 2023 •
edited

Hi @vinokurig @olexii4,

Please have a look at the picture below - that is how I believe the git services page may look like:

Mockups

mockup_git_services

cc @ibuziuk

@olexii4 olexii4 mentioned this issue Nov 2, 2023
Merged
@ibuziuk ibuziuk mentioned this issue Nov 23, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@olexii4 olexii4

Labels
area/dashboard kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P1 Has a major impact to usage or development of the system. team/A This team is responsible for the Che Operator and all its operands as well as chectl and Hosted Che
Projects
None yet
Milestone
No milestone
5 participants
@olexii4 @vinokurig @amisevsk @akurinnoy @che-bot