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

Hide strategy error messages for non-admins #821

Closed
kenkunz opened this issue Sep 19, 2024 · 0 comments · Fixed by #822
Closed

Hide strategy error messages for non-admins #821

kenkunz opened this issue Sep 19, 2024 · 0 comments · Fixed by #822
Assignees
Labels
enhancement New feature or request priority: P1 An important issue that must be fixed before the next release, but does nto require immediate focus. size: S A small task that can be completed in less than a day

Comments

@kenkunz
Copy link
Contributor

kenkunz commented Sep 19, 2024

Goal

As a non-admin user viewing strategies, I don't want to see error messages that are only relevant to technical users, so that I am not distracted unnecessarily by aspects of strategy execution that are not relevant to me.

Acceptance criteria

  1. For a non-admin user viewing strategy index, a strategy with errors should not include an [Error] badge in the strategy tile (with the exception of strategies that failed to return a valid metadata response).
  2. For non-admin user viewing the strategy overview page of a strategy with an error, the error alert should not be displayed.
  3. For an admin user viewing strategy index, a strategy with errors should include the [Error] badge as it does today. In addition, the tooltip should include a note that the error message is only displayed for admin users.
  4. For admin user viewing the strategy overview page of a strategy with an error, the error alert should continue to be displayed. In addition, it should include a note that the error is only displayed for admin users.
@kenkunz kenkunz added enhancement New feature or request priority: P1 An important issue that must be fixed before the next release, but does nto require immediate focus. size: S A small task that can be completed in less than a day labels Sep 19, 2024
@kenkunz kenkunz self-assigned this Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority: P1 An important issue that must be fixed before the next release, but does nto require immediate focus. size: S A small task that can be completed in less than a day
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant