-
-
Notifications
You must be signed in to change notification settings - Fork 658
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
feat: update dependency permission #4910
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -30,14 +32,16 @@ export const DependencyRow: FC<{ feature: IFeatureToggle }> = ({ feature }) => { | |||
<FlexRow> | |||
<StyledDetail> | |||
<StyledLabel>Dependency:</StyledLabel> | |||
<Button | |||
startIcon={<Add />} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
startIcon was breaking permission button disabled state
@@ -1,6 +1,7 @@ | |||
export const ADMIN = 'ADMIN'; | |||
export const CREATE_FEATURE = 'CREATE_FEATURE'; | |||
export const UPDATE_FEATURE = 'UPDATE_FEATURE'; | |||
export const UPDATE_DEPENDENCY = 'UPDATE_DEPENDENCY'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we call it UPDATE_FEATURE_DEPENDENCY
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure, updating across all files to keep it consistent.
SELECT assign_unleash_permission_to_role('UPDATE_FEATURE_DEPENDENCY', 'Member'); | ||
SELECT assign_unleash_permission_to_role('UPDATE_FEATURE_DEPENDENCY', 'Owner'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, I didn't know you could do this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
About the changes
Add without permission
Add with permission (had to remove plus icon since it breaks the without permission state with lock - only one icon is possible)
Edit/delete without permission:
Edit/delete with permission:
Important files
Discussion points