Skip to content

fix(aci): Surface API error detail in edit form save toast#114444

Merged
malwilley merged 1 commit intomasterfrom
malwilley/monitor-save-error-detail
Apr 30, 2026
Merged

fix(aci): Surface API error detail in edit form save toast#114444
malwilley merged 1 commit intomasterfrom
malwilley/monitor-save-error-detail

Conversation

@malwilley
Copy link
Copy Markdown
Member

The edit detector form was always showing a generic "Unable to update monitor" toast when saving failed. We have a util that extracts a useful error message, we just weren't using it.

Made the change for both the new and old form logic, but the old one will be gone soon

The edit detector form always showed a generic 'Unable to update monitor'
message when saving failed, even when the API returned a specific error
like 'You do not have permission to perform this action.' Extract the
response detail using the same getWorkflowEngineResponseErrorMessage
helper already used by the create detector form.
@malwilley malwilley requested a review from a team as a code owner April 30, 2026 16:45
@malwilley malwilley changed the title fix(monitors): Surface API error detail in edit form save toast fix(aci): Surface API error detail in edit form save toast Apr 30, 2026
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 30, 2026
@malwilley malwilley merged commit 633933c into master Apr 30, 2026
73 checks passed
@malwilley malwilley deleted the malwilley/monitor-save-error-detail branch April 30, 2026 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants