Skip to content

Conversation

@jhaaaa
Copy link
Collaborator

@jhaaaa jhaaaa commented Oct 22, 2025

Reword and restructure syncAll documentation in docs/pages/chat-apps/list-stream-sync/sync-and-syncall.mdx to clarify optimizations and scope

Update the syncAll documentation to describe synced items and terminology changes in sync-and-syncall.mdx.

  • Reframe syncAll to list syncing of new welcomes, conversations with unread messages, and preferences
  • Replace “streaming” with “syncing” and document consent states [ALLOWED, UNKNOWN, DENIED]
  • Add return summary details including total conversations and count synced
  • Revise tip to reference group chats a user is no longer part of and standardize “sync” terminology

📍Where to Start

Start with the syncAll section and the bulleted list of synced items in sync-and-syncall.mdx.


📊 Macroscope summarized f374211. 0 files reviewed, 0 issues evaluated, 0 issues filtered, 0 comments posted

🗂️ Filtered Issues

No issues evaluated.

\

Changes since #450 opened

  • Removed outdated documentation about syncAll limitations [f374211]

Macroscope summarized f374211.

@jhaaaa jhaaaa requested a review from a team as a code owner October 22, 2025 21:12
@vercel
Copy link

vercel bot commented Oct 22, 2025

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

Project Deployment Preview Comments Updated (UTC)
docs-xmtp-org Ready Ready Preview Comment Oct 23, 2025 5:42pm


:::tip[Note]
Syncing does not refetch existing conversations and messages. It also does not fetch messages for group chats you are no longer a part of.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We've had this note up top for a while -- I am wondering if we should remove it.

The first sentence is untrue -- or stated incorrectly? I think the info is better handled in the syncAll section specifically. And do we need to state the second sentence?

Copy link
Contributor

Choose a reason for hiding this comment

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

I personally find it confusing. When I really think about it, I can understand what "does not refetch existing conversations and messages" means, but it's not intuitive.

I think we can ditch.

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