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

Improve workflow when user denies Bitbucket access for Che #19274

Closed
sleshchenko opened this issue Mar 11, 2021 · 3 comments
Closed

Improve workflow when user denies Bitbucket access for Che #19274

sleshchenko opened this issue Mar 11, 2021 · 3 comments
Labels
area/che-server kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.
Milestone
7.30

Comments

@sleshchenko
Copy link
Member

sleshchenko commented Mar 11, 2021 •
edited by skabashnyuk
Loading

Is your task related to a problem? Please describe.

Currently after user denies access to Bitbucket server, he gets page with JSON

{
  "message": "401\noauth_problem=permissions_denied"
}

Describe the solution you'd like

But ^ it's pure UX, instead dashboard should be opened where user is notified that they denied Bitbucket access and it's not possible to resolve factory from the specified URL.

Describe alternatives you've considered

Be optimistic, and leave as is keeping in mind that nobody must denies access.

Additional context

Screenshot_20210311_114729
Screenshot_20210311_114749
Bitbucket epic - #18385
@sleshchenko sleshchenko added kind/task Internal things, technical debt, and to-do tasks to be performed. area/che-server area/dashboard labels Mar 11, 2021
@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 Mar 11, 2021
@yhontyk yhontyk added severity/P2 Has a minor but important impact to the 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. labels Mar 12, 2021
@skabashnyuk skabashnyuk mentioned this issue Mar 24, 2021
Closed
11 tasks
@skabashnyuk skabashnyuk added this to the 7.29 milestone Mar 24, 2021
@skabashnyuk skabashnyuk mentioned this issue Mar 25, 2021
Closed
10 tasks
@sleshchenko
Copy link
Member Author

removed area/dashboard. A dedicated issue should be created for dashboard if needed.

@skabashnyuk skabashnyuk mentioned this issue Apr 14, 2021
Closed
12 tasks
@xbaran4 xbaran4 mentioned this issue Apr 19, 2021
Merged
9 tasks
@skabashnyuk skabashnyuk modified the milestones: 7.29, 7.30 Apr 20, 2021
@sleshchenko
Copy link
Member Author

Just for link, Dashboard has a separate issue which we expect to be fixed in 7.30: Dashboard should handle error codes from oauth flow #19722

@sparkoo
Copy link
Member

sparkoo commented May 4, 2021

fixed by #19620 and #19723

@sparkoo sparkoo closed this as completed May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/che-server kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system.
Projects
None yet
Milestone
7.30
Development

No branches or pull requests
5 participants
@sparkoo @skabashnyuk @sleshchenko @yhontyk @che-bot