feat(frontend): refactor state managers to centralize localStorage sync and add ErrorBoundary (#639)#650
Conversation
…nc and add ErrorBoundary
|
@harshitanagpal05 is attempting to deploy a commit to the ritesh Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
…er assertion errors
…istence-error-boundaries
…syntax/ESLint errors
…ency in incident service
Description
This PR implements a centralized state persistence engine and a robust error-handling layer across the frontend architecture to satisfy the requirements of issue #639:
safePersistStorage):authStore,adminStore, andticketStore) in a robust storage wrapper.QuotaExceededErrorwrite exceptions by falling back transparently to a transient in-memory store.persistent-storage-errorevent to track persistence health.ErrorBoundary):AdminHeader.jsxto resolve static analysis alerts.Related Issue
Fixes #639
Type of Change
Checklist
Additional Notes
npm run lintandnpm run buildboth compile with absolute success and zero warnings/errors.closes [BOUNTY] [level:advanced] Refactor React State Managers to Centralize LocalStorage Sync and Error Boundaries #639