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

chore(BIC-1999): couchbase-lite-C 3.1.7 #11

Merged
merged 13 commits into from
Sep 13, 2024
Merged

chore(BIC-1999): couchbase-lite-C 3.1.7 #11

merged 13 commits into from
Sep 13, 2024

Conversation

Nao-ris
Copy link
Contributor

@Nao-ris Nao-ris commented Sep 11, 2024

What's new: https://docs.couchbase.com/couchbase-lite/3.1/cbl-whatsnew.html

Mostly the introduction of scopes and collections, many functions targeting the default scope/collection are now deprecated.

@Nao-ris Nao-ris requested a review from a team as a code owner September 11, 2024 08:48
@Nao-ris Nao-ris changed the title Merge couchbase-lite-C 3.1.7 couchbase-lite-C 3.1.7 Sep 11, 2024
@Nao-ris Nao-ris changed the title couchbase-lite-C 3.1.7 feat(BIC-1999): couchbase-lite-C 3.1.7 Sep 11, 2024
@Nao-ris Nao-ris changed the title feat(BIC-1999): couchbase-lite-C 3.1.7 chore(BIC-1999): couchbase-lite-C 3.1.7 Sep 11, 2024
@xdm67x
Copy link

xdm67x commented Sep 11, 2024

I think we need to check it together tomorrow, if you have time.

@Nao-ris
Copy link
Contributor Author

Nao-ris commented Sep 11, 2024

I think we need to check it together tomorrow, if you have time.

Tomorrow I don't have much time, CSE all day :-)

Comment on lines +654 to +656
#[deprecated(
note = "please use `collection_property_decryptor` on default collection instead"
)]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You deprecated some fields but you don't use them accordingly in the engine side ^^

@Nao-ris Nao-ris merged commit 9191da6 into main Sep 13, 2024
3 of 7 checks passed
@Nao-ris Nao-ris deleted the 3.1.x branch September 13, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants