Skip to content

Commit 418d633

Browse files
guguclaude
andcommitted
Add group prefix to policy action labels for clarity
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 451b572 commit 418d633

2 files changed

Lines changed: 10 additions & 10 deletions

File tree

frontend/src/app/components/users/cedar-policy-list/cedar-policy-list.component.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,8 @@ describe('CedarPolicyListComponent', () => {
117117

118118
it('should return correct action labels', () => {
119119
expect(component.getActionLabel('*')).toBe('Full access (all permissions)');
120-
expect(component.getActionLabel('connection:read')).toBe('Read');
121-
expect(component.getActionLabel('table:edit')).toBe('Edit');
120+
expect(component.getActionLabel('connection:read')).toBe('Connection read');
121+
expect(component.getActionLabel('table:edit')).toBe('Table edit');
122122
expect(component.getActionLabel('table:*')).toBe('Full table access');
123123
});
124124

frontend/src/app/lib/cedar-policy-items.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -24,25 +24,25 @@ export const POLICY_ACTION_GROUPS: PolicyActionGroup[] = [
2424
{
2525
group: 'Connection',
2626
actions: [
27-
{ value: 'connection:read', label: 'Read', needsTable: false },
28-
{ value: 'connection:edit', label: 'Full access', needsTable: false },
27+
{ value: 'connection:read', label: 'Connection read', needsTable: false },
28+
{ value: 'connection:edit', label: 'Connection full access', needsTable: false },
2929
],
3030
},
3131
{
3232
group: 'Group',
3333
actions: [
34-
{ value: 'group:read', label: 'Read', needsTable: false },
35-
{ value: 'group:edit', label: 'Manage', needsTable: false },
34+
{ value: 'group:read', label: 'Group read', needsTable: false },
35+
{ value: 'group:edit', label: 'Group manage', needsTable: false },
3636
],
3737
},
3838
{
3939
group: 'Table',
4040
actions: [
4141
{ value: 'table:*', label: 'Full table access', needsTable: true },
42-
{ value: 'table:read', label: 'Read', needsTable: true },
43-
{ value: 'table:add', label: 'Add', needsTable: true },
44-
{ value: 'table:edit', label: 'Edit', needsTable: true },
45-
{ value: 'table:delete', label: 'Delete', needsTable: true },
42+
{ value: 'table:read', label: 'Table read', needsTable: true },
43+
{ value: 'table:add', label: 'Table add', needsTable: true },
44+
{ value: 'table:edit', label: 'Table edit', needsTable: true },
45+
{ value: 'table:delete', label: 'Table delete', needsTable: true },
4646
],
4747
},
4848
];

0 commit comments

Comments
 (0)