Skip to content

feat: add resource-action pills to custom roles table #14354

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

Merged
merged 11 commits into from
Sep 3, 2024
Merged

feat: add resource-action pills to custom roles table #14354

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
5a61d3f
feat: add resource-action pills to custom roles table
jaaydenh Aug 19, 2024
83c5b09
fix: remove permission from theme and change name to colorRoles
jaaydenh Aug 21, 2024
d78f9a7
fix: revert name from colorRoles to roles
jaaydenh Aug 21, 2024
a7b63ba
fix: format
jaaydenh Aug 30, 2024
437bbd0
fix: custom role with no permissions
jaaydenh Aug 31, 2024
9514f30
feat: extract permissions pull list component and add tests
jaaydenh Aug 31, 2024
9497583
chore: undo color roles name change
jaaydenh Sep 3, 2024
fae3881
feat: add experimental pill colors
jaaydenh Sep 3, 2024
de5e171
fix: format
jaaydenh Sep 3, 2024
af142e3
chore: update experiment name
jaaydenh Sep 3, 2024
1b10a76
chore: cleanup
jaaydenh Sep 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
chore: cleanup
  • Loading branch information
@jaaydenh
jaaydenh committed Sep 3, 2024
commit 1b10a76f0a4c2b366087af984e5ce0ca7d6b6657
12 changes: 5 additions & 7 deletions site/src/pages/ManagementSettingsPage/CustomRolesPage/PermissionPillsList.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { type Interpolation, type Theme, useTheme } from "@emotion/react";
import Stack from "@mui/material/Stack";
import type { Permission, Role } from "api/typesGenerated";
import type { Permission } from "api/typesGenerated";
import { Pill } from "components/Pill/Pill";
import {
Popover,
Expand All @@ -21,7 +21,7 @@ interface PermissionPillsListProps {
export const PermissionPillsList: FC<PermissionPillsListProps> = ({
permissions,
}) => {
const resourceTypes: string[] = getUniqueResourceTypes(permissions);
const resourceTypes = getUniqueResourceTypes(permissions);

return (
<Stack direction="row" spacing={1}>
Expand Down Expand Up @@ -53,11 +53,9 @@ const PermissionsPill: FC<PermissionPillProps> = ({
resource,
permissions,
}) => {
const actions = permissions.filter((p) => {
if (resource === p.resource_type) {
return p.action;
}
});
const actions = permissions.filter(
(p) => resource === p.resource_type && p.action,
);

return (
<Pill css={styles.permissionPill}>
Expand Down
Loading