Skip to content
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

Repo ownership & maintenance status – migrating to react-native-community or a new owner #382

Closed
wamry opened this issue Aug 20, 2019 · 61 comments

Comments

@wamry
Copy link

wamry commented Aug 20, 2019

After going through tons of issues and trying every possible way to get this awful messy repo to work, I tried to work with this "https://github.com/luggit/react-native-config.git#89a602b" and the npm version, it just won't work, does anyone have have a stable fork that i can use ?

i'm using react-native 0.60.5

@azamiya
Copy link

azamiya commented Aug 21, 2019

I'm also looking for that problem.
I tried to use react-native-dotenv, but it says

Can I have more than production & development environment configs?
Sadly, it's not available so far. One of the workaround is generating .env file before triggering RN's bundle script automatically using either shell script or your own custom build pipeline.

@divyanshunegi
Copy link

I am getting the whole app frozen and button press does not work at all in one of the screen after using this library, need any kind of alternate, I am thinking to create a custom Native bridge anyway

@macco-chewy
Copy link

macco-chewy commented Aug 25, 2019

Have you followed the instructions in this issue? #365

@LuizFBrisighello
Copy link

Im on RN 0.60.5 and using react-native-config just to be able to use env:ENVFILE=".env" and react-native-build-config to expose the variables to my js.. nnot the best but its working for now.

@Sathyanarayan09
Copy link

Check this article for an alternative https://medium.com/commutatus/multi-environment-variable-setup-for-react-native-application-70fde4de657f

@atulmy
Copy link

atulmy commented Sep 21, 2019

Considering the popularity and wide adoption, this project should be migrated to https://github.com/react-native-community

@Javi
Copy link

Javi commented Oct 15, 2019

I think this library has been effectively abandoned. Any charitative souls willing to maintain a fork?

@kelset
Copy link

kelset commented Oct 15, 2019

@atulmy I can facilitate making that happen, but the issue is that it seems that the maintainers are not responding to DMs or messages or pings in any way. If they want to reach out to me directly so that we can make this happen, pls DM me on twitter.

@mvrdrew
Copy link
Collaborator

mvrdrew commented Oct 16, 2019

Funny you should say that, I've opened this project:

https://github.com/x-react-native/env-config

Would dearly love some help to get it into shape so I can publish a new npm module asap!

@gorhom
Copy link

gorhom commented Oct 23, 2019

@atulmy I can facilitate making that happen, but the issue is that it seems that the maintainers are not responding to DMs or messages or pings in any way. If they want to reach out to me directly so that we can make this happen, pls DM me on twitter.

@kelset, @atulmy

any updates regarding moving this repo to react-native-community ?

@mvrdrew
Copy link
Collaborator

mvrdrew commented Oct 23, 2019

@gorhom I've been in contact with @kelset about migrating my fork into react-native-community and it's all sounding very promising so far. @kelset is super busy this week though, so he's asked me to remind him about this next week when he has more time.

@gorhom
Copy link

gorhom commented Oct 23, 2019

@melodydrewvr thanks for the update 🎉, let me know if you need help in maintaining it :)

@compojoom
Copy link

@melodydrewvr - any progress?
I've used this library previously on RN < 0.60 and it worked great. Just wanted to use it on a project with RN .61 and read about all the issues.

I was thinking of forking and fixing it for us, but if this is going to move to react-native-community I would love to help with that. @melodydrewvr - if you want let's connect and sync our efforts on this.

@kelset
Copy link

kelset commented Oct 31, 2019

👋 folks - I've asked to the community owners and basically the conversation is about understanding the "overlap" between this lib and the react-native-dotenv alternative.

I've proposed a gradual roadmap that would lead to the two converging, as soon as I have a "final say" of some sort from their side I'll update @melodydrewvr & y'all - hopefully it won't be long.

@compojoom
Copy link

The overlap between react-native-config and react-native-dotenv is only on the javascript side, right?
React-native-config on the other hand allows you to use your ENV variables both in objective-c code or in the java code. Ah, forgot gradle. So it is a little bit more powerful in my opinion.

@pedro
Copy link
Contributor

pedro commented Oct 31, 2019

hey folks! so sorry we dropped the ball here – I'm no longer working at Lugg and they are super busy with other stuff, but we'd all love to work with the community to find a new home and maintainers for this project.

@melodydrewvr @kelset let us know how can we help with the move to react-native-community!

I like how react-native-dotenv works more smoothly in js land, whereas with this plugin you need to recompile the app to see env changes. But being able to use that env in xcode/gradle/etc is really the most important feature here. I'm sure we could make this work better with js files or even merge the two projects if there's someone interested in taking the lead.

Thanks,

@JanithaR
Copy link

JanithaR commented Nov 1, 2019

I am using react-native-config for nearly 3 years now and just a week ago got up on the fence to almost ditch it. And react-native-dotenv was the obvious choice but as @pedro mentioned,

being able to use that env in xcode/gradle/etc is really the most important feature here

so it was a no brainer to say no to react-native-dotenv and somehow get things working and I did.

I'm using my vars in JS, Gradle, Objective-C, Java, XML and Info.plist. The only issue I haven't been able to address is that in Circle CI builds Info.plist vars are always empty so I had to address that by manually entering the needed values for particular schemes as user settings and then refer to that var in the Info.plist file, which is not such a big deal but regardless, it would be awesome to have all configs in just one file.

No doubt this module is very powerful and it works but; most of the pain comes from the confusing documentation on Github and npmjs. Can we at least correct that first? Pretty sure there won't be as much of an outcry from the community if the documentaion is not confusing them in the first place.

@mvrdrew
Copy link
Collaborator

mvrdrew commented Nov 1, 2019

Hi everyone, I'm going to be out of contact for about a week so apologies in advance for not being able to progress the fork until I'm available again. However, if there are any pull requests you'd like to see merged please feel free to raise them in the meantime. In particular, any documentation changes that @JanithaR has mentioned would be a great starting point. I've also gone through the PRs raised against this repo that I'd like to see merged too, so hopefully when I'm free again next week we'll be in a better position to get a new package published (one way or another!)

@pedro
Copy link
Contributor

pedro commented Nov 2, 2019

sounds good, yeah I haven't been following but if anyone can point me to important PR's to merge here I'll give it a try and merge/publish it - at least to stop the bleeding!

@JanithaR
Copy link

JanithaR commented Nov 2, 2019

It's weekend for me and I'm with my kid and wife fulltime. I'll extend my help first thing Monday.

@kelset
Copy link

kelset commented Nov 4, 2019

@pedro how about as step 0 you just migrate this repo to @melodydrewvr (of give him admin rights to the repo & the publishing ones too)?

I'll link this conversation back into the discord.

@pedro
Copy link
Contributor

pedro commented Nov 5, 2019

Definitely, added!

I'll just add a mention of Lugg to the readme, but up to you guys where to take it from here 👌

@pedro pedro pinned this issue Nov 5, 2019
@pedro pedro changed the title Any stable working alternative Repo ownership & maintenance status – migrating to react-native-community or a new owner Nov 5, 2019
@alexpchin
Copy link

alexpchin commented Nov 12, 2019

Hi all, do we have an ETA for this move to react-native-community?

@leethree
Copy link

@pedro could you publish a new version of this library so we can use while waiting for the migration?

@kelset
Copy link

kelset commented Nov 14, 2019

👋 folks - @melodydrewvr was on holiday last week, so it will take a bit longer before this lib will get back on its feet, but a bit of patience will go a long way.

Hi all, do we have an ETA for this move to react-native-community?

No ETA no this for the foreseeable future, if there are updates on that we will keep you updated :)

@pedro
Copy link
Contributor

pedro commented Nov 14, 2019

@leethree oh doh! yes just pushed v0.12.0.

@alexpchin
Copy link

What was included in the v0.12.0 release? There are no notes attached... Is is basically a release of the master branch of this repo, i.e. what was previously:

"react-native-config": "github:luggit/react-native-config#master"

Or were any PRs or changes included? Thanks!

@atulmy
Copy link

atulmy commented Jan 11, 2020

@kelset, can we expect to see this package move to https://github.com/react-native-community soon? @pedro has already given a go ahead to do so in this comment.

May the force be strong with React Native!

Screenshot 2020-01-12 at 1 29 52 AM

@kelset
Copy link

kelset commented Jan 13, 2020

no updates on my side, there was some pushback when I asked internally 😓 I'll try to ask again

@safaiyeh
Copy link

Hi @kelset someone brought this up to me. Not sure the reasoning for the push back, but I wouldn't mind helping maintain the repo when in community.

@JanithaR
Copy link

That would be me. Thanks in advance on behalf of everyone using react-native-config and godspeed. :D

@kelset
Copy link

kelset commented Jan 28, 2020

Hey everyone, I tried to raise this again in the internal Discord but I'm sorry to report that - basically - until this conversation (react-native-community/discussions-and-proposals#176) has reached a final say, there should be no expectation of this repo being moved in the RNCommunity.

What I would recommend for the time being is that the repo & npm rights are given directly to one of the new maintainers.

@compojoom
Copy link

Who are the new maintainers now? @mvrdrew - I don't see a fork under your account?

@mvrdrew
Copy link
Collaborator

mvrdrew commented Mar 17, 2020

I think a lot of interest in updating maintainers died out when a new npm release with the auto-linking included, as that was the biggest blocker a lot of people seemed to have.

I did make a fork under a different account with the intention of getting it into shape for RNC, but after some discussions with @kelset about whether it'd be accepted I lost a bit of momentum. The fork is here: https://github.com/x-react-native/env-config

@compojoom
Copy link

@mvrdrew So what are we going to do? I haven't used the library in an year and I'm looking at it for another project. There are 37 pull requests - which doesn't really give me the confidence that it works.

With react-native-community refusing to accept such packages it leaves us at a very unfortunate place.

We either have to create a b-league as @kelset put it in the discussion on the react-native-community issues or just fork ourselves. I'm against forking myself and adding the pull-requests I'm interested in as this create a disconnected ecosystem. Nobody knows what is current and what not.

@kelset
Copy link

kelset commented Mar 17, 2020

just to clarify folks - I don't think that forking is the solution, but migrating this repo to another user so that it has an active maintainer with all the permissions (both github and npm).

I don't know if @pedro has the correct rights to do so since IIRC admin rights are needed.

@luancurti
Copy link
Collaborator

@erickreutz @pedro Can you help us to migrate to another user that maintain this repo? or give the role (access to commit and publish this package) for someone to volunteer to maintain this repo (I'm one).

Who else volunteers? like this comment to @erickreutz and @pedro (or any repo owner) so they know who the volunteers are

@pedro
Copy link
Contributor

pedro commented Mar 18, 2020

for sure! it's unfortunate that the move to react-native-community got stuck, but happy to give you access so we can keep things going! 👍 just did it, let me know if anything.

What's your npm handle? I can give you permissions to release it too.

@luancurti
Copy link
Collaborator

for sure! it's unfortunate that the move to react-native-community got stuck, but happy to give you access so we can keep things going! 👍 just did it, let me know if anything.

What's your npm handle? I can give you permissions to release it too.

@pedro thanks! my npm https://www.npmjs.com/~luancurti and npm email luancurti@gmail.com

@pedro
Copy link
Contributor

pedro commented Mar 19, 2020

You're the man! Really amazing how you already got to all that stuff!

I'm getting this error trying to add you to the project, any ideas?

$ npm owner add luancurti
npm ERR! code E405
npm ERR! 405 Method Not Allowed - PUT https://registry.npmjs.org/react-native-config/-rev/51-6193bee3061b0569f33fb50db3d00c10

$ npm -v
6.14.2

Can't seem to do it via the web either 🤔

@luancurti
Copy link
Collaborator

@pedro first you need add me as a maintainer after this you can add me as a owner

follow this steps https://docs.npmjs.com/transferring-a-package-from-a-user-account-to-another-user-account

@pedro
Copy link
Contributor

pedro commented Mar 19, 2020

Odd, from the command line it really doesn't work, but after disabling adblock I could see the admin tab and add you from the web – lemme know if it's all good now!

I'll let you decide on newer versions/etc, fwiw we were doing semver and enjoying the flexibilities of major version 0 heh. Feel free to go for 1.0 obv, whatever makes sense!

@luancurti
Copy link
Collaborator

Odd, from the command line it really doesn't work, but after disabling adblock I could see the admin tab and add you from the web – lemme know if it's all good now!

It worked! Thanks!

I'll let you decide on newer versions/etc, fwiw we were doing semver and enjoying the flexibilities of major version 0 heh. Feel free to go for 1.0 obv, whatever makes sense!

Of course, thanks again!

@JanithaR
Copy link

Repo is back in business I see. ;)

@maxkomarychev
Copy link
Contributor

Hey,

I created alternative config library react-native-ultimate-config. It supports all features of react-native-config + a bit more (like exposing typed values and not just strings).

In addition many issues that have been opened recently in react-native-config do not exist in my lib (well, at least as far as I could test them):

#449 , #446 , #438 , #432 , #426, #443, #442, #413 and others of that sort.

@luancurti
Copy link
Collaborator

@maxkomarychev if you would like to help us to fix this issues. Will be a great help and the community will be very grateful

@lesmo
Copy link

lesmo commented Jun 13, 2020

Glad to see progress here!

Seeing that things should go smoother now, @maxkomarychev if you forked from this one it'd be awesome if you pushed new features here. Many, many people are already using this lib including myself. It makes more sense to me for it to be here.

@kelset
Copy link

kelset commented Jun 15, 2020

@luancurti may be worth closing this issue, with you writing a last comment saying how the repo is now again under proper maintenance and leave some way of contacting / reaching out to you if they want to help you maintain it; then lock the issue so that it will be the last comment.

On top of that, echoing what both @luancurti and @lesmo said, @maxkomarychev I would suggest that you join forces and push your work back into this main repo and help by becoming a maintainer of it 🤗 ofc this is OSS so if you prefer to go your own way with the fork noone can blame you for that 🤗

@maxkomarychev
Copy link
Contributor

maxkomarychev commented Jun 15, 2020

@luancurti @lesmo @kelset

Hi, I created react-native-ultimate-config from scratch as it was much easier to avoid common issues by rethinking some design decisions. It is essentially different library. While it's important to avoid fragmentation of community contributing it back in a normal way into react-native-config is not something I find trivial.

That said I'd be happy to help the community. Of course I am biased to my lib but perhaps you would consider a scenario where my lib supersedes react-native-config in order to retain the community and get issues solved. Of course this will be a breaking change and we'd need to create migration guide. And of course this is a viable option as long as my lib is tested well and we make sure there is no degradation.

@maxkomarychev
Copy link
Contributor

to complement my previous answer: if superseding scenario is not an option and you prefer to keep existing ruby scripts and other things I'd still be happy to share my views on design and why I did what I did in my library

@luancurti
Copy link
Collaborator

Thanks to all for the discussion in this issue. I'll close this because we find a new maintainer and we are looking to help to maintain this package them feel free to open pull requests in this repository

@lugg lugg locked as resolved and limited conversation to collaborators Jun 15, 2020
@luancurti luancurti unpinned this issue Jun 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests