Skip to content

Commit 59928ec

Browse files
committed
refactor: split frontend block proposal success & miss notification events
See: BEDS-1079
1 parent 76477f1 commit 59928ec

File tree

6 files changed

+25
-9
lines changed

6 files changed

+25
-9
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
@@ -21,7 +21,8 @@ function closeDialog(): void {
2121
2222
const checkboxes = ref({
2323
is_attestations_missed_subscribed: props.value?.is_attestations_missed_subscribed ?? false,
24-
is_block_proposal_subscribed: props.value?.is_block_proposal_subscribed ?? false,
24+
is_block_proposal_missed_subscribed: props.value?.is_block_proposal_missed_subscribed ?? false,
25+
is_block_proposal_success_subscribed: props.value?.is_block_proposal_success_subscribed ?? false,
2526
is_group_efficiency_below_subscribed: props.value?.is_group_efficiency_below_subscribed ?? false,
2627
is_max_collateral_subscribed: props.value?.is_max_collateral_subscribed ?? false,
2728
is_min_collateral_subscribed: props.value?.is_min_collateral_subscribed ?? false,
@@ -113,8 +114,12 @@ const { minutes: minutesUntilNetworkFinality } = formatSecondsTo(secondsUntilNet
113114
:info="$t('notifications.subscriptions.validators.attestation_missed.info', { count: Number(formatSecondsTo(secondsPerEpoch, { minimumFractionDigits: 1 }).minutes) })"
114115
/>
115116
<BcSettingsRow
116-
v-model:checkbox="checkboxes.is_block_proposal_subscribed"
117-
:label="$t('notifications.subscriptions.validators.block_proposal.label')"
117+
v-model:checkbox="checkboxes.is_block_proposal_success_subscribed"
118+
:label="$t('notifications.subscriptions.validators.block_proposal_success.label')"
119+
/>
120+
<BcSettingsRow
121+
v-model:checkbox="checkboxes.is_block_proposal_missed_subscribed"
122+
:label="$t('notifications.subscriptions.validators.block_proposal_missed.label')"
118123
/>
119124
<BcSettingsRow
120125
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: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -418,7 +418,12 @@
418418
"text": "Indicates the percentage of rewards in relation to the theoretical maximum value",
419419
"title": "Attestation Efficiency: "
420420
},
421-
"block_proposal": "Block Proposal",
421+
"block_proposal_missed": {
422+
"label": "Block proposal missed"
423+
},
424+
"block_proposal_success": {
425+
"label": "Block proposal success"
426+
},
422427
"sync": {
423428
"text": "If isPartOfSyncCommittee then sync_efficiency = executed_Sync / (scheduled_Sync - missed_Blocks) else 1",
424429
"title": "Sync Committee 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)