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 workspace detail #1746

Merged
merged 6 commits into from
Jan 16, 2025
Merged

Add workspace detail #1746

merged 6 commits into from
Jan 16, 2025

Conversation

fhlavac
Copy link
Contributor

@fhlavac fhlavac commented Jan 15, 2025

Description

Added workspace detail with "Role assignments" and "Assets" tabs. Hidden the roles tab under platform.rbac.workspaces-role-bindings

RHCLOUD-31951

Screenshots

Before:

N/A

After:

Snímek obrazovky 2025-01-15 v 17 49 32

Checklist ☑️

  • PR only fixes one issue or story
  • Change reviewed for extraneous code
  • UI best practices adhered to
  • Commits squashed and meaningfully named
  • All PR checks pass locally (build, lint, test, E2E)

  • (Optional) QE: Needs QE attention (OUIA changed, perceived impact to tests, no test coverage)
  • (Optional) QE: Has been mentioned
  • (Optional) UX: Needs UX attention (end user UX modified, missing designs)
  • (Optional) UX: Has been mentioned

@fhlavac fhlavac requested a review from a team as a code owner January 15, 2025 16:53
@fhlavac
Copy link
Contributor Author

fhlavac commented Jan 16, 2025

/retest

@fhlavac
Copy link
Contributor Author

fhlavac commented Jan 16, 2025

/retest

@fhlavac fhlavac closed this Jan 16, 2025
@fhlavac fhlavac reopened this Jan 16, 2025
@karelhala karelhala merged commit 0899d91 into RedHatInsights:master Jan 16, 2025
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@karelhala karelhala karelhala approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fhlavac @karelhala