docs(KNO-12105): update push engagement status copy in message-statuses#1356
Open
docs(KNO-12105): update push engagement status copy in message-statuses#1356
Conversation
Replace outdated "Not currently supported" push entries in the engagement status section with accurate copy. Push read/open status is not implicitly managed by Knock but can be set manually via the message engagement API using the knock_message_id from the push payload. Knock's mobile SDKs handle this automatically on tap. Same correction applied to the link-clicked push entry. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Risk LOW: Updates push engagement status copy in message-statuses.mdx, replacing "Not currently supported" with accurate descriptions of manual and SDK-based status management.
Reasons
- Only a single
.mdxcontent file is modified — no components, layouts, or configuration files touched - The diff is minimal at 2 additions and 2 deletions (4 total lines changed)
- No new pages, sidebar changes, or image additions
- Changes are confined to a single documentation section (
content/send-notifications/) - Pure copy update with no structural or logic changes
Notes
- Verify the references to "message engagement API" and "Knock's mobile SDKs" are accurate and match current product capabilities
- Confirm that the
knock_message_idterminology aligns with the actual push payload field name - Author (
webbjordy) is not yet in the repository contributor list, so skipping auto-approval — a team member should review and approve
Sent by Cursor Automation: Docs PR classifier
cellomatt
reviewed
Mar 18, 2026
Replace incorrect "can manually set link_clicked" copy with accurate explanation: push notifications don't support clickable links so Knock link tracking is unavailable for push. Direct readers to the interacted status with metadata for tap event tracking instead. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
Author
|
Hey @cellomatt and @scotidodson addressed the feedback on the |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Summary
Push — Not currently supportedentry under Marked as read / opened with accurate copy explaining that push read status is not implicitly managed by Knock but can be set manually via the message engagement API usingknock_message_id, and that Knock's mobile SDKs handle this automatically when a notification is tapped.knock_message_idfrom the push payload." (SDK sentence omitted here since tap handlers setread/interacted, notlink_clicked).Closes KNO-12105.
Test plan
docs.knock.app/send-notifications/message-statuses#marked-as-read--openedand#link-clicked🤖 Generated with Claude Code