Skip to content

Conversation

jasonyuezhang
Copy link
Owner

Closes https://linear.app/getsentry/issue/EME-395/improve-error-handling-for-failed-upload-case-not-size-error-but

We should show the message related to the error for the user's improved comprehension

Before:
Screenshot 2025-10-07 at 9 26 10 AM

After:
image


Copied from getsentry#101086
Original PR: getsentry#101086

Copy link

Improve Error Message Display on Artifact Build Details Failure

This PR updates the error handling logic in the BuildDetails page for preprod artifacts. Specifically, it changes the way the error message is selected and displayed to the user when a build details fetch fails. Instead of always displaying a generic fallback message, the UI will now show a specific error message from buildDetailsQuery.error?.responseJSON?.error when available, improving user understanding of upload failures.

Key Changes

• Modified the message prop of the BuildError component in static/app/views/preprod/buildDetails/buildDetails.tsx to prioritize displaying a relevant API error message if present.
• Replaced use of buildDetailsQuery.error?.message with a type check for buildDetailsQuery.error?.responseJSON?.error.
• Maintained the generic fallback message for cases where a specific error is not present.

Affected Areas

static/app/views/preprod/buildDetails/buildDetails.tsx
• Error rendering logic on the preprod build details page

This summary was automatically generated by @propel-code-bot

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