-
-
Notifications
You must be signed in to change notification settings - Fork 27k
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
Feature: Support querystring on npm start #1450
Comments
Hmm. Why not set up a redirect in |
Interesting. I hadn't considered that. The only downsides I see:
The use case: App that is normally called from a separate app with required context provided via querystring. I have the config above working on a fork of create-react-app, but I had to disable the Apple specific alternative to opn in react-dev-utils/openBrowser because it ignores querystrings. |
I think this is a bit of an unusual use case, and I don't think we'll be supporting this by default. |
I think I'm going to close this unless there's more support for this being configurable. Since 0.9.0, you can override "start": "BROWSER=none npm start && node scripts/myOpen.js" |
Seems like this use case will be fixed by #1590. |
Some apps need to be started with querystring parameters. So it would be helpful to specify an optional querystring for npm start.
Suggested approach:
Thoughts?
The text was updated successfully, but these errors were encountered: