Skip to content

Notification rich content#32

Draft
ianmacartney wants to merge 5 commits intomainfrom
cursor/notification-rich-content-9475
Draft

Notification rich content#32
ianmacartney wants to merge 5 commits intomainfrom
cursor/notification-rich-content-9475

Conversation

@ianmacartney
Copy link
Copy Markdown
Member

Add richContent field to the notification schema to support rich media (image, audio, video) in Expo push notifications.


Open in Cursor Open in Web

- Added richContent optional field with image, audio, and video properties
- Updated both the Convex schema validator and TypeScript types
- Follows Expo push notification specification for rich content support

Co-authored-by: ian <ian@convex.dev>
@cursor
Copy link
Copy Markdown

cursor bot commented Jan 23, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 23, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Jan 23, 2026

Open in StackBlitz

npm i https://pkg.pr.new/get-convex/expo-push-notifications/@convex-dev/expo-push-notifications@32

commit: 0d13524

- Updated generated component types to include richContent field
- Added build:codegen npm script
- Upgraded convex package to latest version
- richContent supports image, audio, and video URLs for iOS notifications

Co-authored-by: ian <ian@convex.dev>
@reeceyang
Copy link
Copy Markdown
Contributor

@cursor can you merge in changes from main?

@cursor
Copy link
Copy Markdown

cursor bot commented Jan 29, 2026

❌ Unable to add follow-up after multiple attempts. The agent may be experiencing issues.

Please try again later or Open in Web to view the status.


Open in Cursor Open in Web

Learn more about Cursor Agents

@reeceyang
Copy link
Copy Markdown
Contributor

The docs for the richContent field says:

Currently supports setting a notification image. Provide an object with key image and value of type string, which is the image URL. Android will show the image out of the box. On iOS, you need to add a Notification Service Extension target to your app. See this example on how to do that.

@reeceyang
Copy link
Copy Markdown
Contributor

@cursor can we add any other missing fields? See https://docs.expo.dev/push-notifications/sending-notifications/#message-request-format

Also, the richContent field has type Object - it doesn't specify what the fields are.

@cursor
Copy link
Copy Markdown

cursor bot commented Jan 29, 2026

❌ Unable to add follow-up after multiple attempts. The agent may be experiencing issues.

Please try again later or Open in Web to view the status.


Open in Cursor Open in Web

Learn more about Cursor Agents

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.

3 participants