Skip to content

Fix reminder email recipient company resolution - #1326

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/POR-20533-cursor-automated-triage-response-6d0a
Draft

Fix reminder email recipient company resolution#1326
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/POR-20533-cursor-automated-triage-response-6d0a

Conversation

@cursor

@cursor cursor Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Resolve missing client recipient company IDs from Copilot before enqueueing reminder email dispatches.
  • Require reminder email payloads to include recipientCompanyId instead of omitting it when task data is missing.
  • Add focused tests for the missing-company fallback and stricter reminder email payload.

Testing Criteria

  • yarn test src/jobs/notifications/send-task-reminders.test.ts src/jobs/notifications/send-reminder-email.test.ts src/jobs/notifications/dispatch-reminder-email.test.ts (22 tests passed)
  • yarn lint:check (passed)
  • yarn tsc attempted, but fails on existing SVG module resolution errors in src/icons/index.ts; no notification-related type errors were reported before the SVG failures.

Notes

  • Linear and Sentry MCP servers required authentication in the automation environment, so the Sentry event details could not be fetched directly. The webhook reported the issue in Triage, and this repo is configured for Sentry project tasks.

Impact & Surface Area of Change

  • Affects scheduled task reminder email dispatch only. The change prevents Copilot notification email calls for client recipients from being sent without a recipient company, which is required by the app's existing notification recipient invariant and likely triggers portal notification sender lookup failures.
Open in Web View Automation 

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

linear-code Bot commented Jun 18, 2026

Copy link
Copy Markdown

POR-20533

@vercel

vercel Bot commented Jun 18, 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, Comment Jun 18, 2026 12:17am

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