xmtp / xmtp-js-content-types

MIT License
8 stars 6 forks source link

fix: use type from the reply nested content directly #21

Closed dmccartney closed 1 year ago

dmccartney commented 1 year ago

This gets the nested type of a reply from the deserialized EncodedContent instead of inspecting the parameter map.

See also https://github.com/xmtp/xmtp-ios/pull/143 https://github.com/xmtp/xmtp-android/pull/104

This is all part of https://github.com/xmtp/xmtp-react-native/pull/81

changeset-bot[bot] commented 1 year ago

🦋 Changeset detected

Latest commit: ae4fcdaa68ea5b1f4430d14f02f73231e194fd57

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

This PR includes changesets to release 1 package | Name | Type | | ------------------------ | ----- | | @xmtp/content-type-reply | 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

rygine commented 1 year ago

we also need to add a changeset to this PR so that the changes are published to NPM. there's a link above in the comment by changeset-bot that should help.