Skip to content

Commit c641ca6

Browse files
committed
refactor: split frontend block proposal success & miss notification events
See: BEDS-1079
1 parent 90bc8d4 commit c641ca6

File tree

6 files changed

+24
-10
lines changed

6 files changed

+24
-10
lines changed

frontend/components/notifications/management/NotificationsManagementDashboards.vue

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,8 +107,11 @@ const wrappedDashboards: ComputedRef<
107107
if (settingsValidatorDashboard.is_attestations_missed_subscribed) {
108108
result.push($t('notifications.subscriptions.validators.attestation_missed.label'))
109109
}
110-
if (settingsValidatorDashboard.is_block_proposal_subscribed) {
111-
result.push($t('notifications.subscriptions.validators.block_proposal.label'))
110+
if (settingsValidatorDashboard.is_block_proposal_missed_subscribed) {
111+
result.push($t('notifications.subscriptions.validators.block_proposal_missed.label'))
112+
}
113+
if (settingsValidatorDashboard.is_block_proposal_success_subscribed) {
114+
result.push($t('notifications.subscriptions.validators.block_proposal_success.label'))
112115
}
113116
if (settingsValidatorDashboard.is_upcoming_block_proposal_subscribed) {
114117
result.push($t('notifications.subscriptions.validators.upcoming_block_proposal.label'))

frontend/components/notifications/management/NotificationsManagementSubscriptionDialog.vue

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,8 @@ function closeDialog(): void {
2020
2121
const checkboxes = ref({
2222
is_attestations_missed_subscribed: props.value?.is_attestations_missed_subscribed ?? false,
23-
is_block_proposal_subscribed: props.value?.is_block_proposal_subscribed ?? false,
23+
is_block_proposal_missed_subscribed: props.value?.is_block_proposal_missed_subscribed ?? false,
24+
is_block_proposal_success_subscribed: props.value?.is_block_proposal_success_subscribed ?? false,
2425
is_group_efficiency_below_subscribed: props.value?.is_group_efficiency_below_subscribed ?? false,
2526
is_max_collateral_subscribed: props.value?.is_max_collateral_subscribed ?? false,
2627
is_min_collateral_subscribed: props.value?.is_min_collateral_subscribed ?? false,
@@ -108,8 +109,12 @@ watch(hasAllEvents, () => {
108109
:info="$t('notifications.subscriptions.validators.attestation_missed.info', { count: Number(formatSecondsTo(secondsPerEpoch, { minimumFractionDigits: 1 }).minutes) })"
109110
/>
110111
<BcSettingsRow
111-
v-model:checkbox="checkboxes.is_block_proposal_subscribed"
112-
:label="$t('notifications.subscriptions.validators.block_proposal.label')"
112+
v-model:checkbox="checkboxes.is_block_proposal_success_subscribed"
113+
:label="$t('notifications.subscriptions.validators.block_proposal_success.label')"
114+
/>
115+
<BcSettingsRow
116+
v-model:checkbox="checkboxes.is_block_proposal_missed_subscribed"
117+
:label="$t('notifications.subscriptions.validators.block_proposal_missed.label')"
113118
/>
114119
<BcSettingsRow
115120
v-model:checkbox="checkboxes.is_upcoming_block_proposal_subscribed"

frontend/components/playground/PlaygroundDialog.vue

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@ const openQuestion = (yesLabel?: string, noLabel?: string) => {
3333
const validatorSub: NotificationSettingsValidatorDashboard = {
3434
group_efficiency_below_threshold: 0,
3535
is_attestations_missed_subscribed: true,
36-
is_block_proposal_subscribed: true,
36+
is_block_proposal_missed_subscribed: true,
37+
is_block_proposal_success_subscribed: true,
3738
is_group_efficiency_below_subscribed: true,
3839
is_max_collateral_subscribed: false,
3940
is_min_collateral_subscribed: false,

frontend/composables/notifications/useNotificationsManagementDashboards.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,8 @@ export function useNotificationsManagementDashboards() {
8585
const accountDashboardSettings = settings as NotificationSettingsValidatorDashboard
8686
accountDashboardSettings.group_efficiency_below_threshold = 0
8787
accountDashboardSettings.is_attestations_missed_subscribed = false
88-
accountDashboardSettings.is_block_proposal_subscribed = false
88+
accountDashboardSettings.is_block_proposal_missed_subscribed = false
89+
accountDashboardSettings.is_block_proposal_success_subscribed = false
8990
accountDashboardSettings.is_group_efficiency_below_subscribed = false
9091
accountDashboardSettings.is_max_collateral_subscribed = false
9192
accountDashboardSettings.is_min_collateral_subscribed = false

frontend/i18n/locales/en.json

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -849,8 +849,11 @@
849849
"info": "We will trigger a notification every epoch ({count} minute) during downtime. | We will trigger a notification every epoch ({count} minutes) during downtime.",
850850
"label": "Attestations missed"
851851
},
852-
"block_proposal": {
853-
"label": "Block proposal (missed & success)"
852+
"block_proposal_missed": {
853+
"label": "Block proposal missed"
854+
},
855+
"block_proposal_success": {
856+
"label": "Block proposal success"
854857
},
855858
"explanation": "All notifications are sent after network finality (~20min).",
856859
"group_efficiency": {

frontend/types/api/notifications.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,8 @@ export interface NotificationSettingsValidatorDashboard {
182182
is_group_efficiency_below_subscribed: boolean;
183183
group_efficiency_below_threshold: number /* float64 */;
184184
is_attestations_missed_subscribed: boolean;
185-
is_block_proposal_subscribed: boolean;
185+
is_block_proposal_success_subscribed: boolean;
186+
is_block_proposal_missed_subscribed: boolean;
186187
is_upcoming_block_proposal_subscribed: boolean;
187188
is_sync_subscribed: boolean;
188189
is_withdrawal_processed_subscribed: boolean;

0 commit comments

Comments
 (0)