-
-
Notifications
You must be signed in to change notification settings - Fork 84
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
API adapters (react-native-tcp, net, tls) #51
Comments
@Overtorment, thanks for your adapters! I will take a look at them. Could you please open a PR so the diff is more readable for me? |
I can do this next week, or meanwhile you can take a look at my PR, and look for 2 files: |
fyi, we migrated and rolling out to production. FYI had to do some changes in adapters https://github.com/BlueWallet/BlueWallet/blob/master/blue_modules/net.js |
@Overtorment, sorry about the releases, I am finally getting some free time to fix and update the package! My objective is to make it as similar as possible to the node net API. The last update 4.0.0, might make it a lot easier for you. |
if it is not improving performance a lot or fixes some major issues I wont update, for now. busy with other stuff, but good job! will plan update on roadmap |
Moved to #41. |
We are slowly migrating to this plugin (pr here BlueWallet/BlueWallet#1223) instead of old
@aprock react-native-tcp
. So far so good! But we will test extensively.I wanted to raise a concern that this package API does not conform not to nodejs net/tls nor to widely used
@aprock react-native-tcp
. It's no big deal, I wrote adapters, but this package cant work as a drop-in replacement out of the box. So I was thinking, maybe it's worth including adapters I wrote (https://github.com/BlueWallet/BlueWallet/pull/1223/files#diff-2054cc4c657c92ae593a544fe4dd140d) to this package so it can be a drop-in replacement?The text was updated successfully, but these errors were encountered: