Improve unauthorized error message
Problem
I've noticed that people always confused with Failed to get completions. Request failed (code 401)
error message.
It's always about missing Duo Pro for a user.
Suggestion
Extend the error message with more details explaining to double check Duo Pro subscription and permissions for a user. Additionally, provide a link to the documentation page.
Designs
- Show closed items
Relates to
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Matt Nohr added workflowready for development label
added workflowready for development label
- Matt Nohr marked this issue as related to gitlab-org/gitlab#461815 (closed)
marked this issue as related to gitlab-org/gitlab#461815 (closed)
- Jon Glassman added UI text label
added UI text label
- Vitali Tatarintev mentioned in issue #33 (closed)
mentioned in issue #33 (closed)
Marking this as a Deliverable for %17.1
- Matt Nohr added Deliverable label
added Deliverable label
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#17795 (closed)
mentioned in issue gitlab-org/quality/triage-reports#17795 (closed)
- Marvin Gay assigned to @marvingay
assigned to @marvingay
- Author Maintainer
I haven't done it myself before, but I had an idea to try wrap and unwrap errors. We can return some error from the call and unwrap it on the higher level, check for a specific error, and provide a user friendly message.
Here some links on that topic:
1 - 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#17901 (closed)
mentioned in issue gitlab-org/quality/triage-reports#17901 (closed)
- Matt Nohr mentioned in epic gitlab-org&14051 (closed)
mentioned in epic gitlab-org&14051 (closed)
- Matt Nohr added to epic gitlab-org&14052
added to epic gitlab-org&14052
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#18066 (closed)
mentioned in issue gitlab-org/quality/triage-reports#18066 (closed)
- Maintainer
This issue is scheduled for completion in this milestone but is not yet in development. Changing health status to 'needs attention'.
Issue participants are welcome to override this by setting the health status to another value.
- 🤖 GitLab Bot 🤖 changed health status to needs attention
changed health status to needs attention
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#18165 (closed)
mentioned in issue gitlab-org/quality/triage-reports#18165 (closed)
- Matt Nohr unassigned @marvingay
unassigned @marvingay
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#18249 (closed)
mentioned in issue gitlab-org/quality/triage-reports#18249 (closed)
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#18358 (closed)
mentioned in issue gitlab-org/quality/triage-reports#18358 (closed)
- Tian Gao mentioned in issue gitlab-org/gitlab#460908 (closed)
mentioned in issue gitlab-org/gitlab#460908 (closed)
- Maintainer
related:
gitlab-org/gitlab#460908 (comment 1943145298)
gitlab-org/gitlab#461815 (closed)
MR !72 (diffs)
Edited by Tian Gao - Tian Gao assigned to @tgao3701908
assigned to @tgao3701908
- Tian Gao added workflowin dev label and removed workflowready for development label
added workflowin dev label and removed workflowready for development label
- Tian Gao mentioned in merge request !72 (merged)
mentioned in merge request !72 (merged)
- Tian Gao mentioned in merge request gitlab-org/gitlab!157607 (merged)
mentioned in merge request gitlab-org/gitlab!157607 (merged)
- 🤖 GitLab Bot 🤖 mentioned in issue gitlab-org/quality/triage-reports#18438 (closed)
mentioned in issue gitlab-org/quality/triage-reports#18438 (closed)
- Vitali Tatarintev mentioned in commit 6999699b
mentioned in commit 6999699b
- Vitali Tatarintev closed with merge request !72 (merged)
closed with merge request !72 (merged)
Changing to workflowcomplete based on the engineering workflow
1- Matt Nohr added workflowcomplete label and removed workflowin dev label
added workflowcomplete label and removed workflowin dev label