-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[README]Add Heroku one click deploy button #1150
Conversation
Are you completely sure it is free and stays alive ? Last time I checked I was able to setup an environment but it was only available for a certain period. Maybe I did something wrong. Will check it again! https://RssBridge.TomDings.com/ I am using my own multi-homed release. Hosted from - currently - 17 locations, continuously updated, many additions - other bridges too - and some awesome tools which I needed. Nice feature ... I can request as many feed updates as I want and all systems keep on running stable. Some details : https://www.facebook.com/Social.Rss.Bridges/
|
Created a new account and got this notification : May I ask what I am doing wrong ? |
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.
Thanks for the PR!
Heroku is a new one for me, looks very promising.
That said, find below one point for consideration.
Please also check the two files composer.json
and composer.lock
, they shouldn't be part of this commit as only the app.json
file is required according to https://devcenter.heroku.com/articles/heroku-button
Co-Authored-By: LogMANOriginal <LogMANOriginal@users.noreply.github.com>
That is not a problem. We don't rely on these services, so adding a button is not a problem. If these services disappear we'll simply remove the deploy button. Of course, if they don't provide a free plan it's up to everyone to pay (or use any of the other deploy options). You can find their pricing plans here. https://www.heroku.com/pricing |
Heroku deploy button requires |
I see. That is unfortunate. Is it maybe possible to move all the files to a subfolder (i.e. That way it's easier to tell what the file belongs to. |
You can try clicking that button from my forked repo azdkj532/rss-bridge, because that link will deploy repo you clicked. |
I'll have a look on this. |
https://devcenter.heroku.com/articles/deploying-php#overview |
I thought so. No problem though, since the filename is distinguishable. |
https://devcenter.heroku.com/articles/heroku-button#requirements |
That is very unfortunate. They should really provide some options for this, as Can we add custom elements to app.json? |
It works! Demo Site |
That is very good news, thanks for taking care of it 👍 One more question, since we have to keep the composer files. Should they require additional extensions to fully support all the requirements? Right now there is only Found some more details about what I mean. https://getcomposer.org/doc/01-basic-usage.md#platform-packages |
Co-Authored-By: LogMANOriginal <LogMANOriginal@users.noreply.github.com>
Some extensions are enabled by default, it's ok to add them explicitly. Thanks. |
Thanks for the updates. I tested it on my machine and found two additional points.
|
This should do it. Thanks again 🥇 |
* Add deploy button to Heroku * Add composer.json and composer.lock (required by Heroku)
Heroku provides free plan for anyone, this commit makes it more easily to deploy a rss-bridge instance.