Skip to content

fix: prevent crash when status report has no updates#2005

Merged
mxkaske merged 1 commit intomainfrom
fix/status-report-empty-updates-crash
Mar 23, 2026
Merged

fix: prevent crash when status report has no updates#2005
mxkaske merged 1 commit intomainfrom
fix/status-report-empty-updates-crash

Conversation

@mxkaske
Copy link
Copy Markdown
Member

@mxkaske mxkaske commented Mar 23, 2026

Summary

  • Dashboard and status page crash when a status report exists but all its updates have been deleted
  • Added null-safe access (?.) for update array elements across 5 files
  • Falls back to report.createdAt for date display when no updates exist

🤖 Generated with Claude Code

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openstatus-dashboard Ready Ready Preview, Comment Mar 23, 2026 0:20am
openstatus-status-page Ready Ready Preview, Comment Mar 23, 2026 0:20am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
openstatus-web Skipped Skipped Mar 23, 2026 0:20am

Request Review

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 6 files

Status reports with all updates deleted would crash both the dashboard
and status page due to unguarded array access. Falls back to
report.createdAt when no updates exist.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@mxkaske mxkaske force-pushed the fix/status-report-empty-updates-crash branch from 0194e91 to be2931f Compare March 23, 2026 12:16
@vercel vercel bot temporarily deployed to Preview – openstatus-web March 23, 2026 12:16 Inactive
@mxkaske mxkaske merged commit 7476fd5 into main Mar 23, 2026
12 checks passed
@mxkaske mxkaske deleted the fix/status-report-empty-updates-crash branch March 23, 2026 12:28
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