Skip to content

Add contextual error screen that preserves dashboard layout#79

Merged
stylessh merged 1 commit intomainfrom
stylessh/error-screen-layout
Apr 12, 2026
Merged

Add contextual error screen that preserves dashboard layout#79
stylessh merged 1 commit intomainfrom
stylessh/error-screen-layout

Conversation

@stylessh
Copy link
Copy Markdown
Owner

Summary

  • Error screens now render inside the dashboard card content area instead of replacing the entire shell, preserving the topbar, card chrome, and bottombar.
  • Errors show friendly, contextual messages based on type: 403 opens a "Configure access" prompt via the existing GitHub access dialog, 404 shows "Not found", plus handling for rate limits, network failures, and timeouts.
  • GitHub's raw error detail is shown in a muted code box (with the HTTP method/URL suffix stripped).
  • Renames remaining QuickHub references to DiffKit (user agent, review dismiss message).

Test plan

  • Trigger a 403 error (e.g. access a repo without app installation) — should show "Access not configured" with "Configure access" button that opens the GitHub access dialog
  • Trigger a 404 error — should show "Not found" with search icon
  • Verify dashboard layout (topbar, card, bottombar) remains intact during any error
  • Verify "Try again" button resets the error and re-fetches

Error screens now render inside the card content area instead of
replacing the entire dashboard shell. Errors show friendly messages
based on type (403 → configure access prompt, 404, rate limit,
network, timeout) with GitHub's error detail. Also renames QuickHub
references to DiffKit.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
diffkit a7c64ed Commit Preview URL

Branch Preview URL
Apr 12 2026, 08:07 PM

@stylessh stylessh merged commit 3e408c4 into main Apr 12, 2026
4 checks passed
stylessh added a commit that referenced this pull request Apr 18, 2026
Error screens now render inside the card content area instead of
replacing the entire dashboard shell. Errors show friendly messages
based on type (403 → configure access prompt, 404, rate limit,
network, timeout) with GitHub's error detail. Also renames QuickHub
references to DiffKit.
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