refactor: split frontend block proposal success & miss notification e…

…vents See: BEDS-1079
gobitfly · Jan 15, 2025 · 38da3ff · 38da3ff
1 parent 1b58ca9
commit 38da3ff
Show file tree

Hide file tree

Showing 6 changed files with 25 additions and 9 deletions.
diff --git a/frontend/components/notifications/management/NotificationsManagementDashboards.vue b/frontend/components/notifications/management/NotificationsManagementDashboards.vue
@@ -107,8 +107,11 @@ const wrappedDashboards: ComputedRef<
     if (settingsValidatorDashboard.is_attestations_missed_subscribed) {
       result.push($t('notifications.subscriptions.validators.attestation_missed.label'))
     }
-    if (settingsValidatorDashboard.is_block_proposal_subscribed) {
-      result.push($t('notifications.subscriptions.validators.block_proposal.label'))
+    if (settingsValidatorDashboard.is_block_proposal_missed_subscribed) {
+      result.push($t('notifications.subscriptions.validators.block_proposal_missed.label'))
+    }
+    if (settingsValidatorDashboard.is_block_proposal_success_subscribed) {
+      result.push($t('notifications.subscriptions.validators.block_proposal_success.label'))
     }
     if (settingsValidatorDashboard.is_upcoming_block_proposal_subscribed) {
       result.push($t('notifications.subscriptions.validators.upcoming_block_proposal.label'))

diff --git a/frontend/components/notifications/management/NotificationsManagementSubscriptionDialog.vue b/frontend/components/notifications/management/NotificationsManagementSubscriptionDialog.vue
@@ -21,7 +21,8 @@ function closeDialog(): void {
 
 const checkboxes = ref({
   is_attestations_missed_subscribed: props.value?.is_attestations_missed_subscribed ?? false,
-  is_block_proposal_subscribed: props.value?.is_block_proposal_subscribed ?? false,
+  is_block_proposal_missed_subscribed: props.value?.is_block_proposal_missed_subscribed ?? false,
+  is_block_proposal_success_subscribed: props.value?.is_block_proposal_success_subscribed ?? false,
   is_group_efficiency_below_subscribed: props.value?.is_group_efficiency_below_subscribed ?? false,
   is_max_collateral_subscribed: props.value?.is_max_collateral_subscribed ?? false,
   is_min_collateral_subscribed: props.value?.is_min_collateral_subscribed ?? false,
@@ -113,8 +114,12 @@ const { minutes: minutesUntilNetworkFinality } = formatSecondsTo(secondsUntilNet
           :info="$t('notifications.subscriptions.validators.attestation_missed.info', { count: Number(formatSecondsTo(secondsPerEpoch, { minimumFractionDigits: 1 }).minutes) })"
         />
         <BcSettingsRow
-          v-model:checkbox="checkboxes.is_block_proposal_subscribed"
-          :label="$t('notifications.subscriptions.validators.block_proposal.label')"
+          v-model:checkbox="checkboxes.is_block_proposal_success_subscribed"
+          :label="$t('notifications.subscriptions.validators.block_proposal_success.label')"
+        />
+        <BcSettingsRow
+          v-model:checkbox="checkboxes.is_block_proposal_missed_subscribed"
+          :label="$t('notifications.subscriptions.validators.block_proposal_missed.label')"
         />
         <BcSettingsRow
           v-model:checkbox="checkboxes.is_upcoming_block_proposal_subscribed"

diff --git a/frontend/components/playground/PlaygroundDialog.vue b/frontend/components/playground/PlaygroundDialog.vue
@@ -33,7 +33,8 @@ const openQuestion = (yesLabel?: string, noLabel?: string) => {
 const validatorSub: NotificationSettingsValidatorDashboard = {
   group_efficiency_below_threshold: 0,
   is_attestations_missed_subscribed: true,
-  is_block_proposal_subscribed: true,
+  is_block_proposal_missed_subscribed: true,
+  is_block_proposal_success_subscribed: true,
   is_group_efficiency_below_subscribed: true,
   is_max_collateral_subscribed: false,
   is_min_collateral_subscribed: false,

diff --git a/frontend/composables/notifications/useNotificationsManagementDashboards.ts b/frontend/composables/notifications/useNotificationsManagementDashboards.ts
@@ -85,7 +85,8 @@ export function useNotificationsManagementDashboards() {
     const accountDashboardSettings = settings as NotificationSettingsValidatorDashboard
     accountDashboardSettings.group_efficiency_below_threshold = 0
     accountDashboardSettings.is_attestations_missed_subscribed = false
-    accountDashboardSettings.is_block_proposal_subscribed = false
+    accountDashboardSettings.is_block_proposal_missed_subscribed = false
+    accountDashboardSettings.is_block_proposal_success_subscribed = false
     accountDashboardSettings.is_group_efficiency_below_subscribed = false
     accountDashboardSettings.is_max_collateral_subscribed = false
     accountDashboardSettings.is_min_collateral_subscribed = false

diff --git a/frontend/i18n/locales/en.json b/frontend/i18n/locales/en.json
@@ -418,7 +418,12 @@
                     "text": "Indicates the percentage of rewards in relation to the theoretical maximum value",
                     "title": "Attestation Efficiency: "
                 },
-                "block_proposal": "Block Proposal",
+                "block_proposal_missed": {
+                    "label": "Block proposal missed"
+                },
+                "block_proposal_success": {
+                    "label": "Block proposal success"
+                },
                 "sync": {
                     "text": "If isPartOfSyncCommittee then sync_efficiency = executed_Sync / (scheduled_Sync - missed_Blocks) else 1",
                     "title": "Sync Committee Efficiency: "

diff --git a/frontend/types/api/notifications.ts b/frontend/types/api/notifications.ts
@@ -182,7 +182,8 @@ export interface NotificationSettingsValidatorDashboard {
   is_group_efficiency_below_subscribed: boolean;
   group_efficiency_below_threshold: number /* float64 */;
   is_attestations_missed_subscribed: boolean;
-  is_block_proposal_subscribed: boolean;
+  is_block_proposal_success_subscribed: boolean;
+  is_block_proposal_missed_subscribed: boolean;
   is_upcoming_block_proposal_subscribed: boolean;
   is_sync_subscribed: boolean;
   is_withdrawal_processed_subscribed: boolean;