Skip to content

fix(POR-22656): handle platform Price.prices.amount unmarshaling errors in notification-center - #1422

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/por-22656-cursor-automated-triage-response-cf89
Draft

fix(POR-22656): handle platform Price.prices.amount unmarshaling errors in notification-center#1422
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/por-22656-cursor-automated-triage-response-cf89

Conversation

@cursor

@cursor cursor Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Wrap /notification-center notification fetch in try/catch so platform API failures (including Go Price.prices.amount JSON unmarshaling errors) show a user-friendly SilentError instead of an unhandled request error in Sentry.
  • Return early when token payload lacks a valid notificationId UUID.
  • Use safeParse for CTA params with a clear message when the notification is not task-linked.

Testing Criteria

  • ESLint passes on src/app/notification-center/page.tsx
  • Change mirrors the defensive pattern used in POR-22642/22643/22646+ fixes
  • Manual: open an in-product notification that triggers a platform Price.prices.amount unmarshaling error and confirm the page shows "This notification could not be opened in Tasks" instead of a 500

Notes

  • Linear: POR-22656
  • Underlying root cause is a platform backend schema mismatch (Price.prices.amount stored as string vs Go int64). Platform team should fix the data/model; this PR only prevents Tasks from surfacing it as an unhandled error.

Impact & Surface Area of Change

  • /notification-center page only — error handling for notification deep-link flow when platform API fails.
Open in Web View Automation 

…rs in notification-center

Co-authored-by: Neil Raina <makeitraina@users.noreply.github.com>
@linear-code

linear-code Bot commented Aug 22, 2026

Copy link
Copy Markdown

POR-22656

@vercel

vercel Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
tasks-app Ready Ready Preview Aug 22, 2026 4:34am

Request Review

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