react-native-network-info icon indicating copy to clipboard operation
react-native-network-info copied to clipboard

Merging react-native-network-info with react-native-netinfo

Open Rapsssito opened this issue 5 years ago • 12 comments

As commented here: https://github.com/react-native-community/react-native-netinfo/issues/52#issuecomment-527646754 by @mikehardy, these two libraries seem like an API surface area duplication and potential to perhaps have multiple active maintainers in one repo?

Is there any possibility of merging both repositories, @pusherman and @matt-oakes, into one so we get the best of them?

Rapsssito avatar Sep 29 '19 08:09 Rapsssito

👋 I would be really happy to merge these two libraries together.

@pusherman Let me know if you would also be happy to collaborate and we can talk about how this would work in practice.

matt-oakes avatar Sep 29 '19 10:09 matt-oakes

awsome merge I would like to help you too

hosseinmd avatar Sep 29 '19 11:09 hosseinmd

Yeah, me too!

Rapsssito avatar Sep 29 '19 11:09 Rapsssito

@matt-oakes please begin and tell us what to do

hosseinmd avatar Sep 29 '19 11:09 hosseinmd

Hey @matt-oakes, I think merging the two may make sense. I'll shoot you an email using the email address from your website. We can update this thread once we have a plan.

pusherman avatar Sep 29 '19 12:09 pusherman

@pusherman or @matt-oakes any news?

Rapsssito avatar Nov 05 '19 19:11 Rapsssito

@pusherman I’ve not heard from you yet. Feel free to get in touch if you’re still interested in this

matt-oakes avatar Nov 17 '19 17:11 matt-oakes

Hey @matt-oakes I sent an email to the email on your website ([email protected]) on Oct 1st. I'll resent it now. Let me know if you don't get it or if there's a better email address to reach you at.

pusherman avatar Nov 17 '19 20:11 pusherman

@pusherman or @matt-oakes, any news?

Rapsssito avatar Feb 03 '20 16:02 Rapsssito

@Rapsssito - I did talk with @matt-oakes about this. The plan is to open up a PR on the react-native-netinfo side that merges in the functionality from this library. I have not had time to get started on it yet but if anyone else here wants to give it a go-- that's the direction we will head.

pusherman avatar Feb 03 '20 16:02 pusherman

@pusherman, glad to hear that! Could you open an issue over react-native-netinfo with a TODO and refer to this issue?

I will close this issue once that issue is created.

Rapsssito avatar Feb 03 '20 16:02 Rapsssito

This would be a great help! We recently tried to implement react-native-network-info to try to get default gateway address but had to retreat to react-native-netinfo after running into some issues. Would be wonderful to have the best of both worlds!

ependergrass avatar Sep 02 '20 19:09 ependergrass