Skip to content

Conversation

@kkrypt0nn
Copy link
Contributor

Heroku no longer offers a free plan.

@netlify
Copy link

netlify bot commented Dec 3, 2024

Deploy Preview for pydis-static ready!

Name Link
🔨 Latest commit e9e7fb9
🔍 Latest deploy log https://app.netlify.com/sites/pydis-static/deploys/674efb9312d54f00088834fa
😎 Deploy Preview https://deploy-preview-1444--pydis-static.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@coveralls
Copy link

Coverage Status

coverage: 100.0%. remained the same
when pulling e9e7fb9 on kkrypt0nn:patch-heroku-vps-guide
into 4d4f816 on python-discord:main.

Copy link

@shenanigansd shenanigansd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Though all these criticisms are still valid on the paid tiers, aren't they? As a possible alternative, we could change the header from "why free hosts are bad" to "why shared hosts are bad", and leave the content intact.

@kkrypt0nn
Copy link
Contributor Author

kkrypt0nn commented Dec 3, 2024

"Shared hosts" is maybe not the right term, any VPS is technically shared as long as you don't have a dedicated server.

Though open to the idea to merge from replit & heroku everything under a subtitle

@jchristgit
Copy link
Member

I'm a bit 50/50 on this, because as said above, these criticisms still stand.
I'm not sure we should merge this as is.

@kkrypt0nn
Copy link
Contributor Author

kkrypt0nn commented Feb 7, 2025

I do believe that no points of Heroku are valid for a title about shared hosts - just the last point of Replit. Could move the last point of Replit to a subtitle like "Why shared IP hosts are bad". It would just contain a single point though. (Maybe can be written as a small note?)

Alternatively remove the entire "Replit" subtitle, and just put in the bullet points for free hosts, listing Replit as an example. The bullet points would contain the ones of Heroku as well. Though those bullet points won't apply for all free hosting solutions.

Copy link
Contributor

@wookie184 wookie184 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this change is fine as-is. If someone wanted to rewrite the section to just include a list of common issues instead of doing it per service, or to add in more current free services, I think that would be fine, but that seems separate from the issue this PR is fixing.

@kkrypt0nn
Copy link
Contributor Author

I think this change is fine as-is. If someone wanted to rewrite the section to just include a list of common issues instead of doing it per service, or to add in more current free services, I think that would be fine, but that seems separate from the issue this PR is fixing.

I definitely could do that if wished, also renaming the PR

@wookie184
Copy link
Contributor

Sure, sounds good.

@jchristgit jchristgit merged commit af681db into python-discord:main Mar 2, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants