From f0e69cf3749f133a22544cce2b0bb7964c0146df Mon Sep 17 00:00:00 2001 From: maria-hambardzumian Date: Fri, 26 Apr 2024 11:54:06 +0400 Subject: [PATCH] EPMRPP 90569 || Code review fixes --- .../content/notifications/ruleGroup/ruleGroup.jsx | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/app/src/pages/inside/projectSettingsPageContainer/content/notifications/ruleGroup/ruleGroup.jsx b/app/src/pages/inside/projectSettingsPageContainer/content/notifications/ruleGroup/ruleGroup.jsx index cb1f660c90..424ac20377 100644 --- a/app/src/pages/inside/projectSettingsPageContainer/content/notifications/ruleGroup/ruleGroup.jsx +++ b/app/src/pages/inside/projectSettingsPageContainer/content/notifications/ruleGroup/ruleGroup.jsx @@ -93,8 +93,8 @@ export const RuleGroup = ({ pluginName, typedRules, notifications, isPluginEnabl projectPluginNotificationsStateSelector(pluginName), ); - const isAvailable = canUpdateSettings(userRole, projectRole); - const isReadOnly = !isAvailable || !isPluginEnabled; + const isUpdateSettingAvailable = canUpdateSettings(userRole, projectRole); + const isReadOnly = !isUpdateSettingAvailable || !isPluginEnabled; const onToggleHandler = (isEnabled, notification) => { trackEvent(PROJECT_SETTINGS_NOTIFICATIONS_EVENTS.SWITCH_NOTIFICATION_RULE(isEnabled)); @@ -246,7 +246,7 @@ export const RuleGroup = ({ pluginName, typedRules, notifications, isPluginEnabl - {isAvailable && + {isUpdateSettingAvailable && (!isPluginEnabled ? (

@@ -299,7 +299,7 @@ export const RuleGroup = ({ pluginName, typedRules, notifications, isPluginEnabl dataAutomationId="notificationsRulesList" className={cx('rule-group-list')} /> - {isAvailable && ( + {isUpdateSettingAvailable && (