Skip to content

Handle network errors in useSubmissionsList test run fetch#8475

Merged
dem4ron merged 1 commit into
mainfrom
fix/8467
Feb 9, 2026
Merged

Handle network errors in useSubmissionsList test run fetch#8475
dem4ron merged 1 commit into
mainfrom
fix/8467

Conversation

@iHiD
Copy link
Copy Markdown
Member

@iHiD iHiD commented Feb 9, 2026

Closes #8465
Closes #8467

Summary

  • Add .catch() handler to the test run fetch in useSubmissionsList that removes the submission from the list on network failure, resetting the UI to a clean state instead of leaving it stuck in a loading/pending state
  • The Sentry beforeSend filter already drops "Failed to fetch" errors (commit 741e5e7), so no new Sentry filtering is needed
  • TypeError: Failed to fetch (exercism.org) #8465 (request-query.ts) needs no code change — React Query already handles fetch errors via its built-in retry/error state

cc @dem4ron

Test plan

  • yarn test passes (160/160 suites, 1550/1550 tests)
  • Verify in browser: when a network error occurs during test run fetch, the editor resets to a clean state rather than showing a stuck loading indicator

🤖 Generated with Claude Code

When fetching test run data fails due to a transient network error
(TypeError: Failed to fetch), remove the submission from the list
to reset the UI to a clean state instead of leaving it stuck.

Closes #8465
Closes #8467

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dem4ron dem4ron marked this pull request as ready for review February 9, 2026 13:40
@dem4ron dem4ron self-requested a review as a code owner February 9, 2026 13:40
@dem4ron dem4ron merged commit 91600eb into main Feb 9, 2026
41 checks passed
@dem4ron dem4ron deleted the fix/8467 branch February 9, 2026 14:03
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.

TypeError: Failed to fetch (exercism.org) TypeError: Failed to fetch (exercism.org)

2 participants