-
Notifications
You must be signed in to change notification settings - Fork 1
Description
[P1] Handle delete errors so confirmation dialog can close
The confirmation dialog is only dismissed when either cancelDelete$ fires or deleteResult$ emits. Because deleteResult$ subscribes directly to accountingService.delete without error handling, any failure in that request will propagate an error into the merged confirmItem$ stream, terminating the setConfirmItem subscription. In that state the dialog stays open and further cancel/confirm clicks no longer update confirmItem, effectively blocking the UI until reload. Consider catching errors from the delete observable and mapping them to a dismissal (and optionally a toast) so the dialog always closes and the stream remains subscribed.
Useful? React with 👍 / 👎.
Originally posted by @chatgpt-codex-connector[bot] in #70 (comment)