Skip to content
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

Edits to the documentation #553

Draft
wants to merge 14 commits into
base: master
Choose a base branch
from
Draft

Conversation

PadowYT2
Copy link
Contributor

Description for commits

There are going to be more updates in a small amount of time, so this will be as a draft

As CentOS 8 is EOL, it should be not supported by Pterodactyl and replaced by Rocky Linux 8 & 9 and AlmaLinux 8 & 9. The question is, how should it look like?

https://cdn.padow.ru/8sNRu

or

https://cdn.padow.ru/6OKuQ

@@ -1,9 +1,7 @@
server {
# Replace the example <domain> with your domain name or IP address
Copy link
Contributor

Choose a reason for hiding this comment

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

This shouldn't be removed IMO. Yes, it's a bit redundant but many users easily overlook <domain> so there is no harm in leaving it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just looks weird without having the text in configs older than PHP 8.1, bu alright

Copy link
Contributor Author

Choose a reason for hiding this comment

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

although, should be this added into SSL configs aswell as Apache? I still don't see a point in this comment

@PadowYT2 PadowYT2 requested a review from Boy132 October 19, 2023 19:00
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.

None yet

3 participants