-
Notifications
You must be signed in to change notification settings - Fork 23
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
refactor: use react-native-quick-crypto, remove rn-nodify #1392
Conversation
ios/Podfile.lock
Outdated
- MMKV (1.3.1): | ||
- MMKVCore (~> 1.3.1) | ||
- MMKVCore (1.3.1) | ||
- MMKV (1.3.0): | ||
- MMKVCore (~> 1.3.0) | ||
- MMKVCore (1.3.0) |
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'm qurious why MMKV version changed? I don't have it on my local maching when I do pod install
maybe you should try pod install --repo-update
?
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.
done, but now it updated it to newer versions. Not sure why it always downgrades some packages on my machine.
ccc8705
to
6e360e8
Compare
6e360e8
to
ab3a962
Compare
Description
Replaces
crypto
calls withreact-native-quick-crypto
which should offer way better performance since it's using JSI/C++. Allows us to removern-nodify
and for Node core module polyfilling. Also replacestext-encoding
with faster alternativefastestsmallesttextencoderdecoder
. Renames.node-version
file so it can be picked up by version managers.Type of change
Tests
QA Notes
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce during the Bitkit testing session. You can also leave a video of the PR in action.