Skip to content

fix(InstallWizard): surface structured API errors on install creation#1601

Merged
dionlow merged 1 commit intomainfrom
fix/install-wizard-error-handling
Apr 28, 2026
Merged

fix(InstallWizard): surface structured API errors on install creation#1601
dionlow merged 1 commit intomainfrom
fix/install-wizard-error-handling

Conversation

@dionlow
Copy link
Copy Markdown
Contributor

@dionlow dionlow commented Apr 27, 2026

Summary

The error handling for createInstallation was not being deconstructed by the standardized handleServerError function.

before

Screenshot 2026-04-27 at 6 35 50 PM

after

Screenshot 2026-04-27 at 6 35 32 PM

The Review step's create-installation onError handler called
setSubmissionError(error.message), which only contains the SDK's generic
"Response returned an error code" string. The standardized API problem
document (with causes / detail / remedy) was discarded, so users hit
errors like "1 destination(s) not found: [testWebhook]" but only saw
the generic message.

Route the install error through handleServerError(), the existing
project-wide utility (already used in Configure, ConnectionsContextProvider,
InstallIntegrationContextProvider, UninstallButton, etc.) that parses the
problem-document body and joins causes / falls back to detail / appends
remedy. This brings the InstallWizard in line with the rest of the
codebase's error-handling pattern.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@dionlow dionlow marked this pull request as ready for review April 27, 2026 23:37
@dionlow dionlow merged commit ffa42bb into main Apr 28, 2026
4 checks passed
@dionlow dionlow deleted the fix/install-wizard-error-handling branch April 28, 2026 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants