From effb23a46d720d309fa71c1d6fee97f76c6abf88 Mon Sep 17 00:00:00 2001 From: Matthew Raymer Date: Sun, 6 Jul 2025 12:41:48 +0000 Subject: [PATCH] refactor: standardize notification usage in ChoiceButtonDialog.vue Add notify helper property and initialize in created() with createNotifyHelpers(this.$notify) Replace direct $notify call in open() with this.notify.confirm(...) and use TIMEOUTS.MODAL for duration Remove unused NotificationIface import Add $notify property for Vue runtime injection to satisfy type checker Ensure all notification usage is consistent with project best practices This ensures maintainable, type-safe, and consistent notification handling in the ChoiceButtonDialog component. --- doc/migration-progress-tracker.md | 4 ++-- src/components/ChoiceButtonDialog.vue | 28 +++++++++++---------------- 2 files changed, 13 insertions(+), 19 deletions(-) diff --git a/doc/migration-progress-tracker.md b/doc/migration-progress-tracker.md index 1246e2dd..31fbc1d9 100644 --- a/doc/migration-progress-tracker.md +++ b/doc/migration-progress-tracker.md @@ -191,8 +191,8 @@ export default class ComponentName extends Vue { **Progress**: 3/15 (20%) - [x] UserNameDialog.vue ✅ **MIGRATED** -- [ ] ActivityListItem.vue -- [ ] AmountInput.vue +- [x] AmountInput.vue ✅ **REVIEWED (no migration needed)** + - Pure UI component, no databaseUtil or notification usage. - [ ] ChoiceButtonDialog.vue - [ ] ContactNameDialog.vue - [ ] DataExportSection.vue diff --git a/src/components/ChoiceButtonDialog.vue b/src/components/ChoiceButtonDialog.vue index 241a1b73..80080cc4 100644 --- a/src/components/ChoiceButtonDialog.vue +++ b/src/components/ChoiceButtonDialog.vue @@ -66,11 +66,12 @@