Skip to content

Added <OakErrorBoundary/> component - #764

Merged
orangemug merged 3 commits into
mainfrom
feat/OC-32-error-boundary
Aug 24, 2026
Merged

Added <OakErrorBoundary/> component#764
orangemug merged 3 commits into
mainfrom
feat/OC-32-error-boundary

Conversation

@orangemug

@orangemug orangemug commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

How to review this PR

Leave this text block for the reviewer

  • Check component hierarchy is followed correctly
  • Check the design Heuristics have been followed
  • Check naming conventions have been applied
  • Check for these gotchyas:
    • Missing exports for Oak components
    • Accidental export of Internal components
    • Snapshots of unexpected components have been modified
    • Circular dependencies
    • Code duplication (via not using base components)
    • Non-functional storybook for this or related components
    • Sensitve files changed eg. atoms, or style tokens
    • Relative imports
    • Default exports

Add your PR description below

Added a new <OakErrorBoundary/> component, to allow us to catch error and not crash the page

Link to the design doc

https://www.figma.com/design/YcWQMMhHPVVmc47cHHEEAl/Oak-Design-Kit?node-id=23664-3905&m=dev

A link to the component in the deployment preview

https://oak-components-storybook-git-feat-oc-32-error-boundary.vercel.thenational.academy/?path=/docs/components-messaging-and-feedback-oakerrorboundary--docs

Testing instructions

Trigger the errors in storybook and check the correct thing happens. Also check designs against figma

@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
oak-components-storybook Ready Ready Preview Aug 19, 2026 11:17am

Request Review

@orangemug
orangemug force-pushed the feat/OC-32-error-boundary branch from 736f3c3 to b042c1b Compare August 19, 2026 10:34
@orangemug
orangemug marked this pull request as ready for review August 19, 2026 10:35
@orangemug
orangemug requested review from messmike and njblore August 19, 2026 10:36

@njblore njblore left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Beautiful 🤌

@sonarqubecloud

Copy link
Copy Markdown

@orangemug
orangemug merged commit 89d210a into main Aug 24, 2026
9 checks passed
@orangemug
orangemug deleted the feat/OC-32-error-boundary branch August 24, 2026 15:02
@notion-workspace

Copy link
Copy Markdown

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.

3 participants