Skip to content

fix(slot): prefer graceful degradation when slot not found#114547

Merged
natemoo-re merged 5 commits intomasterfrom
nm/fix/slot-notfound
May 1, 2026
Merged

fix(slot): prefer graceful degradation when slot not found#114547
natemoo-re merged 5 commits intomasterfrom
nm/fix/slot-notfound

Conversation

@natemoo-re
Copy link
Copy Markdown
Member

@natemoo-re natemoo-re commented May 1, 2026

By switching to graceful degradation (render nothing outside of context), we no longer see the "SlotContext not found" error captured by parent ErrorBoundaries during dev because of HMR

However, considering that we've seen this class of error in prod, we should capture an issue in Sentry when this happens.

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 1, 2026
@natemoo-re natemoo-re marked this pull request as ready for review May 1, 2026 14:41
@natemoo-re natemoo-re requested a review from a team as a code owner May 1, 2026 14:41
@natemoo-re natemoo-re marked this pull request as draft May 1, 2026 14:48
@natemoo-re natemoo-re marked this pull request as ready for review May 1, 2026 15:59
@natemoo-re natemoo-re enabled auto-merge (squash) May 1, 2026 16:09
@natemoo-re natemoo-re merged commit 6688c4c into master May 1, 2026
70 checks passed
@natemoo-re natemoo-re deleted the nm/fix/slot-notfound branch May 1, 2026 16:11
cleptric pushed a commit that referenced this pull request May 5, 2026
By switching to graceful degradation (render nothing outside of
context), we no longer see the "SlotContext not found" error captured by
parent ErrorBoundaries during `dev` because of HMR

However, considering that we've seen this class of error in prod, we
should capture an issue in Sentry when this happens.
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