Skip to content

Improve feedback to user when objects aren't retrieved due to RBAC #3667

Merged
foot merged 38 commits intomainfrom
rbac-error-feedback
Aug 29, 2023
Merged

Improve feedback to user when objects aren't retrieved due to RBAC #3667
foot merged 38 commits intomainfrom
rbac-error-feedback

Conversation

@AlinaGoaga
Copy link
Copy Markdown
Contributor

@AlinaGoaga AlinaGoaga commented May 4, 2023

Closes #2427

What changed?
We're doing a pilot on the Application table to display an info note containing the clusters and namespaces that have been searched to retrieved the data.

Why was this change made?
Sometimes users don't understand why they cannot see certain objects in the UI. This can be caused by a lack of permissions. We're looking to include more information on which clusters and namespaces have been searched trying to retrieve objects for a specific page.

How was this change implemented?
ListObjects now returns in addition to the Objects and associated Errors, a list of searched Namespaces (including cluster name and an array of namespaces per cluster)

Screenshot 2023-05-08 at 18 48 40

@AlinaGoaga AlinaGoaga added the type/enhancement New feature or request label May 4, 2023
@AlinaGoaga AlinaGoaga changed the title Add searched namespaces to list objects response Improve feedback to user when objects aren't retrieved due to RBAC May 5, 2023
@AlinaGoaga AlinaGoaga marked this pull request as ready for review May 8, 2023 15:55
@AlinaGoaga AlinaGoaga requested a review from foot May 8, 2023 15:56
@bigkevmcd
Copy link
Copy Markdown
Contributor

@foot Another one for you when you get back :-)

Comment thread ui/hooks/automations.ts Outdated
@foot
Copy link
Copy Markdown
Contributor

foot commented Aug 14, 2023

Tests are hanging 🤔

@foot foot merged commit efbd3ff into main Aug 29, 2023
@foot foot deleted the rbac-error-feedback branch August 29, 2023 12:19
This was referenced Aug 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants