You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Maybe we just remove the "Deploy to Heroku" button for a configured repo? Or somehow check to see if the repo is public before asking if they want the button?
Maybe we just remove the "Deploy to Heroku" button for a configured repo?
I actually like this idea. The original README serves as a primer for getting started with Bullet Train, so it makes sense to me that developers would look at the README and apply any additional configuration (like these deploy buttons) if it's necessary for their use-case, even after the initial setup. We don't necessarily need to have the buttons exist in a new application, as long as we point developers in the right direction.
It may be beneficial to just have solid documentation on how to deploy with each service and link to it in the README too. That would also help to cut down on things to do in bin/configure which would make it quicker/smoother for developers to get setup with a new application.
If we do want the buttons though, I submitted #1614 so developers don't have to write in their repository link.
The text was updated successfully, but these errors were encountered: