xmtp / xmtp-js

XMTP client SDKs for JavaScript applications.

https://xmtp.org/docs

210 stars 38 forks source link

Add streamAllMessages to Conversations #622

Closed rygine closed 3 months ago

rygine commented 3 months ago

Summary

Upgraded to latest node bindings
Added streamAllMessages to Conversations class
Added tests for streamAllMessages
Removed reference to Conversation instance in DecodedMessage

Notes

The Conversation reference was removed from DecodedMessage because it would add too much overhead to retrieve it when streaming all messages as getting access to the underlying NapiGroup would requiring syncing with the network and then iterating over the group list to find it.

changeset-bot[bot] commented 3 months ago

🦋 Changeset detected

Latest commit: 6dd6a0e3bb8a1619e0c7fefe17f2917c57a2712b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

| Name | Type | | ---------------- | ----- | | @xmtp/mls-client | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

cloudflare-pages[bot] commented 3 months ago

Deploying xmtp-js-docs with Cloudflare Pages

Latest commit:	`6dd6a0e`
Status:	✅ Deploy successful!
Preview URL:	https://d75a2269.xmtp-js.pages.dev
Branch Preview URL:	https://rygine-stream-all-messages.xmtp-js.pages.dev

View logs