Skip to content

Commit 8a535b1

Browse files
committed
fix(site): drop redundant default-roles prop JSDoc
The comments either named an experiment that will go stale when `minimum-implicit-member` is promoted, or described implementation rather than behavior. Prop names already convey intent.
1 parent c0ddabe commit 8a535b1

1 file changed

Lines changed: 0 additions & 9 deletions

File tree

site/src/pages/OrganizationSettingsPage/CustomRolesPage/CustomRolesPageView.tsx

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,16 +39,7 @@ interface CustomRolesPageViewProps {
3939
canDeleteOrgRole: boolean;
4040
canEditDefaultRoles: boolean;
4141
isCustomRolesEnabled: boolean;
42-
/**
43-
* Whether to render the Default Roles section. Gated on the
44-
* minimum-implicit-member experiment at the page level.
45-
*/
4642
defaultRolesEnabled?: boolean;
47-
/**
48-
* Whether the deployment is entitled to multi-org features. The
49-
* underlying PATCH endpoint is multi-org gated, so the edit button
50-
* is disabled when this is false.
51-
*/
5243
defaultRolesEntitled?: boolean;
5344
availableOrgRoles?: AssignableRoles[];
5445
onUpdateDefaultRoles?: (roles: string[]) => Promise<void>;

0 commit comments

Comments
 (0)