Skip to content

Conversation

@youngkidwarrior
Copy link
Collaborator

  • Add notification testing CLI script with payload examples
  • Add deployment checklist for notification system
  • Add code review document with findings
  • Update ALLOWED_DATA_KEYS to include sendtag, action, url fields
  • Fix payload field names in test script to match NotificationData types
  • Fix env var names in checklist (TEMPORAL_ADDR, NEXT_PUBLIC_SUPABASE_*)

Co-Authored-By: Claude noreply@anthropic.com

- Add notification testing CLI script with payload examples
- Add deployment checklist for notification system
- Add code review document with findings
- Update ALLOWED_DATA_KEYS to include sendtag, action, url fields
- Fix payload field names in test script to match NotificationData types
- Fix env var names in checklist (TEMPORAL_ADDR, NEXT_PUBLIC_SUPABASE_*)

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Collaborator Author

youngkidwarrior commented Jan 18, 2026

@youngkidwarrior youngkidwarrior marked this pull request as ready for review January 18, 2026 08:23
@github-actions
Copy link

Playwright Test Results

No test results available

No test results were found. This could be because tests failed to run or artifacts were not uploaded.

@github-actions
Copy link

Vercel Unique URL: https://sendapp-f3utgf0py-0xsend.vercel.app
Vercel Preview URL: sendapp-feat-notification-tooling-0xsend.vercel.app
Last Commit: 9e7a3f1

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.

2 participants