Skip to content

Conversation

@tyhonchik
Copy link
Contributor

@tyhonchik tyhonchik commented Jan 23, 2026

APP-270: Add errorUtils.serialize() to safely serialize errors without circular references
APP-389: Add missing 'use client' directives to components using hooks (partial fix)
APP-461: Add useSafeTranslations() hook with fallback for error boundaries
APP-453: Handle 404 responses directly in AragonBackendServiceError to avoid JSON parse errors
APP-463: Skip plugins-by-dao and policies API calls on v2 (endpoints not deployed to production)

Description

Fix multiple production errors related to error handling, hydration, and API compatibility.

  • APP-270: Converting circular structure to JSON - errors containing DOM references crashed JSON.stringify
  • APP-389: Hydration errors (partial fix) - missing client directives caused SSR/client mismatch
  • APP-461: useTranslations must be used within Provider - error boundaries crashed outside provider context
  • APP-453: SyntaxError: Unexpected token - 404 responses with plain text "Not Found" failed JSON parsing
  • APP-463: Error parsing response (status=404) - v3-only endpoints called on v2 production

Notes

  • Hydration fix is partial - DateFormat.RELATIVE may still cause mismatches due to SSR/client time differences
  • API version checks have TODO comments for removal once v3 is deployed to production

Type of Change

  • Major: Breaking change (change that would cause existing functionality to not work as expected)
  • Minor: Feature (non-breaking change which adds new functionality)
  • Patch: Enhancement (non-breaking change to an existing feature)
  • Patch: Bug fix (non-breaking change which fixes an issue)

Developer Checklist:

  • Manually smoke tested the functionality in a preview or locally
  • Confirmed there are no new warnings or errors in the browser console
  • (For User Stories only) Double-checked that all Acceptance Criteria are satisfied
  • Confirmed there are no new warnings on automated tests
  • Merged and published any dependent changes in downstream modules
  • Selected the correct base branch
  • Commented the code in hard-to-understand areas
  • Followed the code style guidelines of this project
  • Reviewed that the Files Changed in Github’s UI reflect my intended changes
  • Confirmed the pipeline checks are not failing

Review Checklist:

  • (For User Stories only) Tested in a preview or locally that all Acceptance Criteria are satisfied
  • Confirmed that changes follow the code style guidelines of this project

@tyhonchik tyhonchik requested a review from milosh86 January 23, 2026 15:40
@tyhonchik tyhonchik requested a review from a team as a code owner January 23, 2026 15:40
@github-actions
Copy link

github-actions bot commented Jan 23, 2026

🚀 Preview Deployment: View Here

APP-270: Add errorUtils.serialize() to safely serialize errors without circular references
APP-389: Add missing 'use client' directives to components using hooks (partial fix)
APP-461: Add useSafeTranslations() hook with fallback for error boundaries
APP-453: Handle 404 responses directly in AragonBackendServiceError to avoid JSON parse errors
APP-463: Skip plugins-by-dao and policies API calls on v2 (endpoints not deployed to production)
@tyhonchik tyhonchik force-pushed the fix-sentry-errors-jan-26 branch from 142396d to 780469e Compare January 23, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants