fix: show schedule delete failure dialog

jjoonleo · jjoonleo · commit 87d5d3e8aff3 · 2026-06-26T16:06:05.000+09:00
diff --git a/lib/l10n/app_en.arb b/lib/l10n/app_en.arb
@@ -460,6 +460,14 @@
     "@deleteScheduleConfirmAction": {
         "description": "Button text to confirm schedule deletion"
     },
+    "scheduleDeleteFailedTitle": "Appointment cannot be deleted",
+    "@scheduleDeleteFailedTitle": {
+        "description": "Title shown when schedule deletion is rejected or fails"
+    },
+    "scheduleDeleteFailedDescription": "This appointment can no longer be deleted. Please refresh the calendar and try again if the status changed recently.",
+    "@scheduleDeleteFailedDescription": {
+        "description": "Description shown when schedule deletion is rejected or fails"
+    },
     "deleteFeedbackTitle": "We hope to meet you again with a better service",
     "@deleteFeedbackTitle": {
         "description": "Delete feedback modal title"
diff --git a/lib/l10n/app_ko.arb b/lib/l10n/app_ko.arb
@@ -147,6 +147,8 @@
     "scheduleDeleteConfirmTitle": "정말 약속을 삭제할까요?",
     "scheduleDeleteConfirmDescription": "약속을 삭제하면 다시 되돌릴 수 없어요.",
     "deleteScheduleConfirmAction": "약속 삭제",
+    "scheduleDeleteFailedTitle": "약속을 삭제할 수 없어요",
+    "scheduleDeleteFailedDescription": "이 약속은 더 이상 삭제할 수 없어요. 상태가 방금 바뀐 경우 캘린더를 새로고침한 뒤 다시 확인해주세요.",
     "deleteFeedbackTitle": "더 좋은 서비스로 다시 만나요",
     "deleteFeedbackDescription": "의견 입력은 선택 사항입니다.\n계속 진행하면 계정 탈퇴 요청이 전송됩니다.",
     "deleteFeedbackPlaceholder": "탈퇴하시는 이유를 알려주세요.",
diff --git a/lib/l10n/app_localizations.dart b/lib/l10n/app_localizations.dart
@@ -716,6 +716,18 @@ abstract class AppLocalizations {
   /// **'Delete appointment'**
   String get deleteScheduleConfirmAction;
 
+  /// Title shown when schedule deletion is rejected or fails
+  ///
+  /// In en, this message translates to:
+  /// **'Appointment cannot be deleted'**
+  String get scheduleDeleteFailedTitle;
+
+  /// Description shown when schedule deletion is rejected or fails
+  ///
+  /// In en, this message translates to:
+  /// **'This appointment can no longer be deleted. Please refresh the calendar and try again if the status changed recently.'**
+  String get scheduleDeleteFailedDescription;
+
   /// Delete feedback modal title
   ///
   /// In en, this message translates to:
diff --git a/lib/l10n/app_localizations_en.dart b/lib/l10n/app_localizations_en.dart
@@ -363,6 +363,13 @@ class AppLocalizationsEn extends AppLocalizations {
   @override
   String get deleteScheduleConfirmAction => 'Delete appointment';
 
+  @override
+  String get scheduleDeleteFailedTitle => 'Appointment cannot be deleted';
+
+  @override
+  String get scheduleDeleteFailedDescription =>
+      'This appointment can no longer be deleted. Please refresh the calendar and try again if the status changed recently.';
+
   @override
   String get deleteFeedbackTitle =>
       'We hope to meet you again with a better service';
diff --git a/lib/l10n/app_localizations_ko.dart b/lib/l10n/app_localizations_ko.dart
@@ -339,6 +339,13 @@ class AppLocalizationsKo extends AppLocalizations {
   @override
   String get deleteScheduleConfirmAction => '약속 삭제';
 
+  @override
+  String get scheduleDeleteFailedTitle => '약속을 삭제할 수 없어요';
+
+  @override
+  String get scheduleDeleteFailedDescription =>
+      '이 약속은 더 이상 삭제할 수 없어요. 상태가 방금 바뀐 경우 캘린더를 새로고침한 뒤 다시 확인해주세요.';
+
   @override
   String get deleteFeedbackTitle => '더 좋은 서비스로 다시 만나요';
 
diff --git a/lib/presentation/calendar/bloc/monthly_schedules_bloc.dart b/lib/presentation/calendar/bloc/monthly_schedules_bloc.dart
@@ -3,6 +3,7 @@ import 'dart:async';
 import 'package:equatable/equatable.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:injectable/injectable.dart';
+import 'package:on_time_front/core/dio/api_error_message.dart';
 import 'package:on_time_front/core/logging/app_logger.dart';
 import 'package:on_time_front/domain/entities/preparation_entity.dart';
 import 'package:on_time_front/domain/entities/schedule_entity.dart';
@@ -35,19 +36,15 @@ class MonthlySchedulesBloc
     on<MonthlySchedulesPreparationsPrefetchRequested>(
       _onPreparationsPrefetchRequested,
     );
-    on<MonthlySchedulesPreparationsStreamChanged>(
-      _onPreparationsStreamChanged,
-    );
+    on<MonthlySchedulesPreparationsStreamChanged>(_onPreparationsStreamChanged);
 
-    _preparationSubscription = _streamPreparationsUseCase().listen(
-      (preparations) {
-        add(
-          MonthlySchedulesPreparationsStreamChanged(
-            preparations: preparations,
-          ),
-        );
-      },
-    );
+    _preparationSubscription = _streamPreparationsUseCase().listen((
+      preparations,
+    ) {
+      add(
+        MonthlySchedulesPreparationsStreamChanged(preparations: preparations),
+      );
+    });
   }
 
   final LoadSchedulesForMonthUseCase _loadSchedulesForMonthUseCase;
@@ -85,9 +82,8 @@ class MonthlySchedulesBloc
         _requestVisibleDatePreparationPrefetch(nextState);
         return nextState;
       },
-      onError: (error, stackTrace) => state.copyWith(
-        status: () => MonthlySchedulesStatus.error,
-      ),
+      onError: (error, stackTrace) =>
+          state.copyWith(status: () => MonthlySchedulesStatus.error),
     );
   }
 
@@ -119,12 +115,14 @@ class MonthlySchedulesBloc
           ? event.endDate
           : state.endDate!;
 
-      emit(state.copyWith(
-        status: () => MonthlySchedulesStatus.loading,
-        schedules: () => state.schedules,
-        startDate: () => startDate,
-        endDate: () => endDate,
-      ));
+      emit(
+        state.copyWith(
+          status: () => MonthlySchedulesStatus.loading,
+          schedules: () => state.schedules,
+          startDate: () => startDate,
+          endDate: () => endDate,
+        ),
+      );
 
       try {
         await _loadSchedulesForMonthUseCase(event.date);
@@ -149,9 +147,7 @@ class MonthlySchedulesBloc
         return nextState;
       },
       onError: (error, stackTrace) {
-        return state.copyWith(
-          status: () => MonthlySchedulesStatus.error,
-        );
+        return state.copyWith(status: () => MonthlySchedulesStatus.error);
       },
     );
   }
@@ -160,19 +156,29 @@ class MonthlySchedulesBloc
     MonthlySchedulesScheduleDeleted event,
     Emitter<MonthlySchedulesState> emit,
   ) async {
+    final previousPreparationMap = Map<String, Duration>.from(
+      state.preparationDurationByScheduleId,
+    );
     try {
-      final updatedPreparationMap =
-          Map<String, Duration>.from(state.preparationDurationByScheduleId)
-            ..remove(event.schedule.id);
-      emit(state.copyWith(
-        lastDeletedSchedule: () => event.schedule,
-        preparationDurationByScheduleId: () => updatedPreparationMap,
-      ));
+      final updatedPreparationMap = Map<String, Duration>.from(
+        previousPreparationMap,
+      )..remove(event.schedule.id);
+      emit(
+        state.copyWith(
+          lastDeletedSchedule: () => event.schedule,
+          preparationDurationByScheduleId: () => updatedPreparationMap,
+        ),
+      );
       await _deleteScheduleUseCase(event.schedule);
     } catch (e) {
-      emit(state.copyWith(
-        status: () => MonthlySchedulesStatus.error,
-      ));
+      emit(
+        state.copyWith(
+          lastDeletedSchedule: () => null,
+          preparationDurationByScheduleId: () => previousPreparationMap,
+          deleteFailureMessage: () => ApiErrorMessage.fromException(e),
+          deleteFailureCount: () => state.deleteFailureCount + 1,
+        ),
+      );
     }
   }
 
@@ -183,18 +189,19 @@ class MonthlySchedulesBloc
     try {
       await _loadSchedulesForMonthUseCase(event.date);
     } catch (_) {
-      emit(state.copyWith(
-        status: () => MonthlySchedulesStatus.error,
-      ));
+      emit(state.copyWith(status: () => MonthlySchedulesStatus.error));
     }
   }
 
   void _onVisibleDateChanged(
     MonthlySchedulesVisibleDateChanged event,
     Emitter<MonthlySchedulesState> emit,
   ) {
-    final normalizedDate =
-        DateTime(event.date.year, event.date.month, event.date.day);
+    final normalizedDate = DateTime(
+      event.date.year,
+      event.date.month,
+      event.date.day,
+    );
     final nextState = state.copyWith(visibleDate: () => normalizedDate);
     emit(nextState);
     _requestVisibleDatePreparationPrefetch(nextState);
@@ -221,8 +228,9 @@ class MonthlySchedulesBloc
           // Stream update already has fresher data.
           continue;
         }
-        final preparation =
-            await _getPreparationByScheduleIdUseCase(scheduleId);
+        final preparation = await _getPreparationByScheduleIdUseCase(
+          scheduleId,
+        );
         fetchedDurations[scheduleId] = preparation.totalDuration;
         hasUpdates = true;
       } catch (_) {
@@ -234,11 +242,7 @@ class MonthlySchedulesBloc
       final updatedMap = Map<String, Duration>.from(
         state.preparationDurationByScheduleId,
       )..addAll(fetchedDurations);
-      emit(
-        state.copyWith(
-          preparationDurationByScheduleId: () => updatedMap,
-        ),
-      );
+      emit(state.copyWith(preparationDurationByScheduleId: () => updatedMap));
     }
   }
 
@@ -272,15 +276,12 @@ class MonthlySchedulesBloc
       return;
     }
 
-    emit(
-      state.copyWith(
-        preparationDurationByScheduleId: () => nextDurations,
-      ),
-    );
+    emit(state.copyWith(preparationDurationByScheduleId: () => nextDurations));
   }
 
   void _requestVisibleDatePreparationPrefetch(
-      MonthlySchedulesState sourceState) {
+    MonthlySchedulesState sourceState,
+  ) {
     final visibleDate = sourceState.visibleDate;
     if (visibleDate == null) {
       return;
@@ -292,29 +293,30 @@ class MonthlySchedulesBloc
     if (scheduleIds.isEmpty) {
       return;
     }
-    add(MonthlySchedulesPreparationsPrefetchRequested(
-        scheduleIds: scheduleIds));
+    add(
+      MonthlySchedulesPreparationsPrefetchRequested(scheduleIds: scheduleIds),
+    );
   }
 
   Map<DateTime, List<ScheduleEntity>> _groupSchedulesByDate(
     List<ScheduleEntity> schedules,
   ) {
-    return schedules.fold<Map<DateTime, List<ScheduleEntity>>>(
-      {},
-      (previousValue, element) {
-        final scheduleTime = DateTime(
-          element.scheduleTime.year,
-          element.scheduleTime.month,
-          element.scheduleTime.day,
-        );
-        if (previousValue.containsKey(scheduleTime)) {
-          previousValue[scheduleTime]!.add(element);
-        } else {
-          previousValue[scheduleTime] = [element];
-        }
-        return previousValue;
-      },
-    );
+    return schedules.fold<Map<DateTime, List<ScheduleEntity>>>({}, (
+      previousValue,
+      element,
+    ) {
+      final scheduleTime = DateTime(
+        element.scheduleTime.year,
+        element.scheduleTime.month,
+        element.scheduleTime.day,
+      );
+      if (previousValue.containsKey(scheduleTime)) {
+        previousValue[scheduleTime]!.add(element);
+      } else {
+        previousValue[scheduleTime] = [element];
+      }
+      return previousValue;
+    });
   }
 
   List<String> _getScheduleIdsForDate(
@@ -329,7 +331,8 @@ class MonthlySchedulesBloc
   }
 
   Set<String> _getCachedScheduleIds(
-      Map<DateTime, List<ScheduleEntity>> schedules) {
+    Map<DateTime, List<ScheduleEntity>> schedules,
+  ) {
     final ids = <String>{};
     for (final scheduleList in schedules.values) {
       for (final schedule in scheduleList) {
diff --git a/lib/presentation/calendar/bloc/monthly_schedules_state.dart b/lib/presentation/calendar/bloc/monthly_schedules_state.dart
@@ -3,19 +3,24 @@ part of 'monthly_schedules_bloc.dart';
 enum MonthlySchedulesStatus { initial, loading, success, error }
 
 final class MonthlySchedulesState extends Equatable {
-  const MonthlySchedulesState(
-      {this.status = MonthlySchedulesStatus.initial,
-      this.schedules = const {},
-      this.preparationDurationByScheduleId = const {},
-      this.lastDeletedSchedule,
-      this.startDate,
-      this.endDate,
-      this.visibleDate});
+  const MonthlySchedulesState({
+    this.status = MonthlySchedulesStatus.initial,
+    this.schedules = const {},
+    this.preparationDurationByScheduleId = const {},
+    this.lastDeletedSchedule,
+    this.deleteFailureMessage,
+    this.deleteFailureCount = 0,
+    this.startDate,
+    this.endDate,
+    this.visibleDate,
+  });
 
   final MonthlySchedulesStatus status;
   final Map<DateTime, List<ScheduleEntity>> schedules;
   final Map<String, Duration> preparationDurationByScheduleId;
   final ScheduleEntity? lastDeletedSchedule;
+  final String? deleteFailureMessage;
+  final int deleteFailureCount;
   final DateTime? startDate;
   final DateTime? endDate;
   final DateTime? visibleDate;
@@ -25,6 +30,8 @@ final class MonthlySchedulesState extends Equatable {
     Map<DateTime, List<ScheduleEntity>> Function()? schedules,
     Map<String, Duration> Function()? preparationDurationByScheduleId,
     ScheduleEntity? Function()? lastDeletedSchedule,
+    String? Function()? deleteFailureMessage,
+    int Function()? deleteFailureCount,
     DateTime? Function()? startDate,
     DateTime? Function()? endDate,
     DateTime? Function()? visibleDate,
@@ -38,6 +45,12 @@ final class MonthlySchedulesState extends Equatable {
       lastDeletedSchedule: lastDeletedSchedule != null
           ? lastDeletedSchedule()
           : this.lastDeletedSchedule,
+      deleteFailureMessage: deleteFailureMessage != null
+          ? deleteFailureMessage()
+          : this.deleteFailureMessage,
+      deleteFailureCount: deleteFailureCount != null
+          ? deleteFailureCount()
+          : this.deleteFailureCount,
       startDate: startDate != null ? startDate() : this.startDate,
       endDate: endDate != null ? endDate() : this.endDate,
       visibleDate: visibleDate != null ? visibleDate() : this.visibleDate,
@@ -46,12 +59,14 @@ final class MonthlySchedulesState extends Equatable {
 
   @override
   List<Object?> get props => [
-        status,
-        schedules,
-        preparationDurationByScheduleId,
-        lastDeletedSchedule,
-        startDate,
-        endDate,
-        visibleDate,
-      ];
+    status,
+    schedules,
+    preparationDurationByScheduleId,
+    lastDeletedSchedule,
+    deleteFailureMessage,
+    deleteFailureCount,
+    startDate,
+    endDate,
+    visibleDate,
+  ];
 }
diff --git a/lib/presentation/calendar/screens/calendar_screen.dart b/lib/presentation/calendar/screens/calendar_screen.dart
diff --git a/test/presentation/calendar/bloc/monthly_schedules_bloc_test.dart b/test/presentation/calendar/bloc/monthly_schedules_bloc_test.dart
diff --git a/test/presentation/calendar/screens/calendar_screen_test.dart b/test/presentation/calendar/screens/calendar_screen_test.dart
