Skip to content

fix: catch fetch errors in displayed and dismissed to prevent unhandled rejections - #330

Open
claude[bot] wants to merge 1 commit into
mainfrom
fix/issue-329-unhandled-rejection
Open

fix: catch fetch errors in displayed and dismissed to prevent unhandled rejections#330
claude[bot] wants to merge 1 commit into
mainfrom
fix/issue-329-unhandled-rejection

Conversation

@claude

@claude claude Bot commented Aug 11, 2026

Copy link
Copy Markdown

Requested by matt · Slack thread

Summary

ApiClient.displayed and ApiClient.dismissed call this.post(...) without handling a rejected promise. When the underlying fetch fails (e.g. network error), this produces an unhandled promise rejection that surfaces in Datadog/Bugsnag.

This mirrors the existing .catch((error) => { console.error(error); }) pattern already used by ApiClient.embed in the same file, applying it to displayed and dismissed as well.

Fixes #329


Generated by Claude Code

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.

displayed / dismissed API calls reject unhandled on network failure

2 participants