-
Notifications
You must be signed in to change notification settings - Fork 678
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
adds config host for ngrok url #802
Conversation
def insert_hosts_into_development_config | ||
inject_into_file( | ||
'config/environments/development.rb', | ||
" config.hosts << /\\h+.ngrok.io/\n", |
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.
We can insert a regex into hosts? I did not know this!
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.
Although the regex is a cool feature, I think we should put in something like ENV['APP_HOST']
so that we can specify it with the .env
file and so that when they deploy they can just change their env file.
4e73ced
to
ec5ab81
Compare
After discussion with @tanema we've decided to keep the regex because not all users will have a host listed in their |
Adds config.hosts required by Rails 6.
Fixes issue brought up in Shopify-App-CLI