-
Notifications
You must be signed in to change notification settings - Fork 63
fix(sg-actions): Remove delete button for system firewall templates #754
Conversation
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.
Please, rename pull request name to reflect the changes
@@ -17,7 +17,7 @@ const SecurityGroupShowRulesAction = { | |||
name: 'SECURITY_GROUP_PAGE.ACTION.RULES', | |||
command: SecurityGroupActionType.View, | |||
icon: 'visibility', | |||
canActivate: (securityGroup: SecurityGroup) => true | |||
canActivate: (securityGroup: SecurityGroup) => securityGroup.type !== SecurityGroupType.PredefinedTemplate |
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.
I believe that the changes should be done 7 lines above. 😕
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.
should we use canActivate or isHidden?
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.
should
@@ -17,7 +17,7 @@ const SecurityGroupShowRulesAction = { | |||
name: 'SECURITY_GROUP_PAGE.ACTION.RULES', | |||
command: SecurityGroupActionType.View, | |||
icon: 'visibility', | |||
canActivate: (securityGroup: SecurityGroup) => true | |||
canActivate: (securityGroup: SecurityGroup) => securityGroup.type !== SecurityGroupType.PredefinedTemplate |
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.
should we use canActivate or isHidden?
b1bf2ed
to
536114c
Compare
No description provided.