Message History: handle Reply better #723
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
neekolas
approved these changes
May 8, 2024
37ng
pushed a commit
that referenced
this pull request
May 10, 2024
nplasterer
added a commit
that referenced
this pull request
May 24, 2024
* Create validated commit for MLS * Create validated commit for MLS * Add/Remove members using GroupMembership * Add/Remove members using GroupMembership * Remove unused import * initial commit * initial commit * Use latest protos * Add missing installations * Validate Credential (#689) * gen protos * trait method * scaffold out validation fn, comment next steps in validation fn * validation of inbox ids * no expiration * key package validation * add files * restore proto * remove println * do not break the wasm bindings * Message History: handle Reply better (#723) * new identity in client (#728) * Consolidate on single method * initial commit * initial commit * new identity in client (#728) * Hack together something that compiles * Fix straggling issues * New DB schema * Stub validated commit v2 * Start on mutable metadata * Add some metadata changes * initial commit * initial commit * new identity in client (#728) * Hack together something that compiles * initial commit * initial commit * new identity in client (#728) * Fix straggling issues * Add inbox_id fixes * Remove dead code * Fix lints * Derive default for MutableMetadataChanges * Rename identity table * Stub validated commit v2 * Identity DB Schema (#739) * New DB schema * Rename identity table * Cache association state (#733) This caches the association state for a given inbox_id and sequence_id, allowing us to skip recomputing it/issuing network requests for smart wallet associations. * Adding and removing group members * More cleanup * Remove after add * Lint * Update xmtp_mls/src/groups/sync.rs Co-authored-by: 37ng <wang7ong@gmail.com> * bindings_ffi: new client & signatureRequest. (#748) * Gather group members from DB * Implement inbox_sequence_id method in client (#737) * Lint * Remove unused import * Get tests passing * Add new GroupUpdated proto * Use new group updated codec * More loose ends * Move back to main * Make pub * More tests passing * Remove a few more todos * Fix up bindings * Fix up bindings * Fix up subscriptions tests * Add hack for unexpected installations * Fix issue with group creation * Fix lints * Use specific docker image * Fix commit validation logic * Update tests since we now have transcript messages * Unignore more tests * Lower concurrency * Change test-threads * Lint * Fix args * Ignore failing test * Update log line * Un-ignore test * Update docker image * Lower number of test threads * Lower number of test threads in bindings tests * Ignore flaky test * Update lock files * Use latest openmls branch * Fix some errors * Fix key package reference * No more compile errors * Remove unused import * Use latest node go * Store key package hash ref * Get inbox ID for address method * Add test * Remove unused import * Lint * Legacy key issues (#767) * add log * add log * bump uniffi version to fix logging issues * force push to hopefully clean CI cache * All String lower case (#777) * lowercase for inbox id * lowercase for in create_client * lint * cargo * bump the openmls version to a merged commit * fix format linter * inbox id not account address anymore * Remove Await Helper in favor of Async Transactions (#779) * remove await helper * do not load mls group on every iteration * don't need to clone provider * keep provider as reference * remove process_for_id_async and just make process_for_id async * bump to the latest jna library * update the makefile with the updated jna * update the binaries --------- Co-authored-by: Andrew Plaza <github.tech@liquidthink.net> Co-authored-by: tuddman <tuddman@users.noreply.github.com> Co-authored-by: yoduyodu <wang7ong@gmail.com> Co-authored-by: Richard Hua <rich@xmtp.com> Co-authored-by: Naomi Plasterer <naomi@xmtp.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
configuration/DELIMITER(rather than the brittle:I hacked in yesterday.send_message_history_request()to return the PIN, as the sending party/device will absolutely need that.Request(MessageHistoryRequest { .. })Reply(MessageHistoryReply { .. })