Skip to content

docs(KNO-12105): update push engagement status copy in message-statuses#1356

Open
webbjordy wants to merge 2 commits intomainfrom
jordy-kno-12105
Open

docs(KNO-12105): update push engagement status copy in message-statuses#1356
webbjordy wants to merge 2 commits intomainfrom
jordy-kno-12105

Conversation

@webbjordy
Copy link
Contributor

Summary

  • Replaces the outdated Push — Not currently supported entry 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 using knock_message_id, and that Knock's mobile SDKs handle this automatically when a notification is tapped.
  • Applies the same correction to the Link clicked push entry — swaps "Not currently supported" for "Not implicitly managed by Knock. You can manually set this status using the message engagement API with the knock_message_id from the push payload." (SDK sentence omitted here since tap handlers set read/interacted, not link_clicked).

Closes KNO-12105.

Test plan

  • Verify the updated push entries read accurately on docs.knock.app/send-notifications/message-statuses#marked-as-read--opened and #link-clicked
  • Confirm no broken links were introduced

🤖 Generated with Claude Code

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>
@webbjordy webbjordy requested review from a team, MikeCarbone and thomaswhyyou and removed request for a team March 18, 2026 06:45
@linear
Copy link

linear bot commented Mar 18, 2026

@vercel
Copy link

vercel bot commented Mar 18, 2026

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

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 18, 2026 9:55pm

Request Review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 .mdx content 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_id terminology 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
Open in Web View Automation 

Sent by Cursor Automation: Docs PR classifier

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>
@webbjordy
Copy link
Contributor Author

webbjordy commented Mar 18, 2026

Hey @cellomatt and @scotidodson addressed the feedback on the link_clicked push entry. Leaving threads 1 and 3 for Scoti's follow-up restructure PR. Ready for another look when you get a chance!

Copy link
Contributor

@scotidodson scotidodson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look great!

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