-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
Make "Deploy to Heroku" button #66
Comments
This is nice to have. |
That would be really awesome. I'm really not super familiar with heroku, I just kind of crashed through getting a Postgres DB up on it and then hosting was less difficult. That being said, this would be a really great option for the Just throwing out ideas to get the ball rolling: Example configuration: module.exports = withHeroku(
withPostgres({ // postgres required for heroku?
pg: {}, // Will default to environment variables that Heroku provides!
heroku: {
appName: 'build-tracker-demo'
},
url: 'https://build-tracker-demo.herokuapp.com'
})
); Then I could see some requirements:
Thoughts? |
PS: Working on more contributing docs that have specifics about plugins as well as a "project vision" to help keep ya'll informed on what went on in my head creating this. @montogeek if you want to get started on this and get stuck, feel free to put up an early PR as a draft or reach out to me on Twitter. |
I can't not longer work on this, feel free to pick it up. |
It would already help a lot to provide an example configuration for heroku or a written guide how to set this up. After digging into heroku and fixing various existing issues I'm presented with plain html in the console after |
I've got a start with a config-less version here: https://github.com/paularmstrong/build-tracker-heroku The down-side to something like this is that you're stuck with an empty artifact configuration. You can't set up any budgets this way. |
Problem
Deploying and getting started requires reading a decent amount of the docs.
Solution
Create "deploy to heroku" button and add to the readme, so folks can quickly spin up a server. If more config is required, maybe a little SPA to generate the config based on some options might be required.
The text was updated successfully, but these errors were encountered: