-
Notifications
You must be signed in to change notification settings - Fork 319
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: move react-native-quick-sqlite to op-sqlite #2541
Conversation
@@ -26,12 +26,12 @@ The following features are currently **NOT** implemented. They will be impleme | |||
|
|||
First and foremost, make sure to follow all steps from [Migrating to v5](../basics/migrating-from-4.x-to-5.x.mdx) guide. To enable offline support, please follow the given steps: |
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.
First and foremost, make sure to follow all steps from [Migrating to v5](../basics/migrating-from-4.x-to-5.x.mdx) guide. To enable offline support, please follow the given steps: | |
First and foremost, make sure to follow all steps from [Migrating to v5](../basics/migrating-from-4.x-to-5.x.mdx) guide. To enable offline support, please follow the given steps: |
this should be changed to v5 to v6. For later PR i think. But mentioning here to not forget.
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.
yes
examples/SampleApp/ios/Podfile
Outdated
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.
this seems like big change.. did you test push notifications after this?
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.
I have removed the changes and added the suggested way as per the official docs of op-sqlite. Updated the troubleshoot section as well in push notifications v2 docs.
Can this be merged so we can move to Expo 51? |
🎯 Goal
The goal of the PR is to move to
op-sqlite
rather thanreact-native-quick-sqlite
for our offline storage as it is much faster and also allows new architecture support.🛠 Implementation details
🎨 UI Changes
iOS
Android
🧪 Testing
☑️ Checklist
develop
branch