Skip to content

Conversation

@Docteh
Copy link
Contributor

@Docteh Docteh commented May 13, 2025

Specifically REVOLT_PUBLIC_URL is for the api, so for these examples should end in /api

Please make sure to check the following tasks before opening and submitting a PR

  • I understand and have followed the contribution guide
  • I have tested my changes locally and they are working as intended

Specifically REVOLT_PUBLIC_URL is for the api, so for these examples should end in /api
@DeclanChidlow
Copy link
Member

Happy with the changes to include /api, but am unsure about the guidance of using the config script. I imagine that most people referring to the additional notes probably have manual configs. They also might not have OpenSSL installed. I'm not sure that it is a necessary change.

@Docteh
Copy link
Contributor Author

Docteh commented Jun 10, 2025

I can take it out, I think I put that in there after reading someone basically would have been happy running it, but didn't know about the script because they skipped the whole section on hostinger.

Yeah just say so and I'll update the PR

@github-project-automation github-project-automation bot moved this from 🆕 Untriaged to 💡 Ready to merge in Pull Request Overview Jul 18, 2025
@insertish
Copy link
Contributor

Duplication is fine here.

@insertish insertish merged commit 21ef057 into stoatchat:master Jul 18, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from 💡 Ready to merge to ✅ Merged in Pull Request Overview Jul 18, 2025
@Docteh Docteh deleted the reedme branch July 18, 2025 11:44
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.

3 participants