-
Notifications
You must be signed in to change notification settings - Fork 19
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: support listBatchMessages (placeholder sequential) #45
Conversation
90d42fc
to
b099fee
Compare
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.
Might be nice to remove the logs in the long run. But looks good to me. 👍
b099fee
to
194cf95
Compare
if (isDebugEnabled) { | ||
Log.v("XMTPModule", msg); | ||
} |
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.
added this in response to #45 (review)
🎉 This PR is included in version 1.2.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
.map { | ||
it.messages(limit = limit, before = beforeDate, after = afterDate) | ||
.map { DecodedMessageWrapper.encode(it) } | ||
} | ||
.flatten() |
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.
You can actually do this with a flatMap
in Kotlin https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.collections/flat-map.html
This is a WIP toward implementing #34.
listBatchMessages
in RN JSTo do in follow-up PRs
batchQuery
so we can implement the performant edition in RN