Skip to content

[8425] It's possible for an error in a single dashlet to break the entire dashboard#4783

Open
jvega190 wants to merge 1 commit intocraftercms:developfrom
jvega190:bugfix/8425
Open

[8425] It's possible for an error in a single dashlet to break the entire dashboard#4783
jvega190 wants to merge 1 commit intocraftercms:developfrom
jvega190:bugfix/8425

Conversation

@jvega190
Copy link
Member

@jvega190 jvega190 commented Feb 3, 2026

craftercms/craftercms#8425

Summary by CodeRabbit

  • Bug Fixes
    • Enhanced dashboard widget error handling to prevent individual widget failures from affecting the entire dashboard display.

@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

Walkthrough

The change imports an ErrorBoundary component and wraps each mapped widget within grid cells with it, adding runtime error boundaries around individual dashboard widgets without modifying public API signatures or behavioral logic.

Changes

Cohort / File(s) Summary
Error Boundary Wrapping
ui/app/src/components/SiteDashboard/SiteDashboard.tsx
Imported ErrorBoundary and wrapped each widget mapper result with error boundaries to isolate failures in individual dashboard widgets.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check ❓ Inconclusive The PR description provides only a ticket reference link without substantive details about the changes, objectives, or testing performed. Expand the description to explain the problem being solved, the solution implemented (ErrorBoundary usage), and any testing performed or considerations.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding error isolation to prevent a single dashlet error from breaking the entire dashboard.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jvega190 jvega190 marked this pull request as ready for review February 3, 2026 22:32
@jvega190 jvega190 requested a review from rart as a code owner February 3, 2026 22:32
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.

1 participant

Comments