-
Notifications
You must be signed in to change notification settings - Fork 413
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: read/unread status #1376
feat: read/unread status #1376
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Co-authored-by: Chris Smith <[email protected]>
…skeleton feat: read/unread status skeleton and kotlin implementation
…skeleton chore: rename
* chore: add api for read unread * chore: remove all flag * chore: add docs for vanilla * chore: change mark as read to promise * Update docs/api/notify/usage.mdx Co-authored-by: Chris Smith <[email protected]> * Update docs/web3inbox/frontend-integration/api.mdx Co-authored-by: Chris Smith <[email protected]> * chore: avoid negatives * chore: add docs for react native --------- Co-authored-by: Chris Smith <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@devceline look good?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Wallet support extracted into a separate PR since it has not been implemented yet: #1535 |
Proposed documentation for the read/unread support in Web3Inbox. This feature has not been implemented yet, but the documentation here should be accurate to what will be built.
Open questions:
Remaining work:
iOS docsComplete changelog page