Skip to content
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

Remove FramesV1 #382

Merged
merged 1 commit into from
Feb 12, 2025
Merged

Remove FramesV1 #382

merged 1 commit into from
Feb 12, 2025

Conversation

nplasterer
Copy link
Contributor

Remove framesV1 from the first class SDK support.

@nplasterer nplasterer self-assigned this Feb 11, 2025
@nplasterer nplasterer marked this pull request as ready for review February 11, 2025 03:40
@nplasterer nplasterer requested a review from a team as a code owner February 11, 2025 03:40
@nplasterer nplasterer merged commit beed257 into np/disappearing-messages Feb 12, 2025
2 of 3 checks passed
@nplasterer nplasterer deleted the np/remove-frames-v1 branch February 12, 2025 05:31
nplasterer added a commit that referenced this pull request Feb 12, 2025
* remove pinned frame url from android

* update libxmtp ref for pinned frame removal and db lock fix

* add message expiration to group and dm

* add ability to update expiration state

* add permission policy update

* write a test for disappearing messages

* remame

* more renames

* small tweaks to impl

* try to get the tests to pass

* try to mirror disappearing message

* try updating test

* update bindings

* update the test

* small test tweak

* getting closer on the test

* add tests for dms as well

* get it passing for dms

* fix up the lint

* rename

* more clean up

* fix lint

* fix lint

* remove small code from test

* remove frames V1 (#382)

* Make addWallet a delicate API (#381)

* add a delicate api warning level

* slightly update the logic

* Update library/src/main/java/org/xmtp/android/library/Client.kt

Co-authored-by: Nicholas Molnar <[email protected]>

* add a test for this case

* fix lint

---------

Co-authored-by: Nicholas Molnar <[email protected]>

* tweak slightly the name

---------

Co-authored-by: Cameron Voell <[email protected]>
Co-authored-by: cameronvoell <[email protected]>
Co-authored-by: Nicholas Molnar <[email protected]>
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.

2 participants