-
Notifications
You must be signed in to change notification settings - Fork 586
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
Upgrade to Realm Core 11.6.0. Was: Adopting to changes in Realm Core (past 11.4.1) #4023
Conversation
7751c72
to
d861347
Compare
84e9581
to
5da103d
Compare
I rebased on latest master. I'll merge as soon as we've got a green build. |
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.
LGTM 👍
FYI, core v11.5.2 had a regression with streaming download notifiers, you may wish to hold of releasing this until the next core version which will include realm/realm-core#5008 |
Using this for my sync branch and it works 👍 |
7f8d658
to
b154e1a
Compare
What, How & Why?
I needed to consume an unreleased version of Realm Core after v11.4.1.
This PR contains the modifications I had to do to adopt to the changes in Realm Core.
Feel free to merge this PR into whatever branch we'll be using when upgrading Realm Core to the version after 11.4.1, once that gets released.
☑️ ToDos
Compatibility
label is updated or copied from previous entryBreaking
label has been applied or is not necessaryIf this PR adds or changes public API's: