Various Changes (see desc) #52
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
StephenDaDev
commented
Nov 16, 2025
StephenDaDev
commented
- Add Choosing a Host Section
- Add Choosing a Host Guide
- Add blank page to use for additional considerations on choosing a host (to be done later)
- Add recommended hosting providers list
- Add unrecommended hosting providers list
- Update community guides to mention replit and northflank issues
- Update threadmenu page to fix formatting concerns
- Add config-references page
- Update config vars page to include new config vars
- Update color names page and move to config references section
- Add Choosing a Host Section - Add Choosing a Host Guide - Add blank page to use for additional considerations on choosing a host (to be done later) - Add recommended hosting providers list - Add unrecommended hosting providers list - Update community guides to mention replit and northflank issues - Update threadmenu page to fix formatting concerns - Add config-references page - Update config vars page to include new config vars - Update color names page and move to config references section
RealCyGuy
previously requested changes
Nov 17, 2025
lorenzo132
previously approved these changes
Nov 19, 2025
Contributor
There was a problem hiding this comment.
Pull Request Overview
This pull request adds comprehensive hosting guidance for Modmail users and reorganizes configuration documentation. The changes include a new "Choosing a Host" guide that helps users select appropriate hosting providers, updates to community guides with warnings about problematic hosts, and an expanded configuration variables reference with detailed categorization.
Key Changes:
- Added new "Choosing a Host" section with guidance on selecting hosting providers, including recommended and unrecommended hosts lists
- Reorganized config documentation into a dedicated "Config References" section with enhanced configuration variables documentation
- Updated community guides with warnings about Replit and Northflank reliability issues
- Fixed formatting issues in the thread menu documentation
Reviewed Changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 19 comments.
Show a summary per file
| File | Description |
|---|---|
choose-host/README.md |
Main hosting selection guide with decision flow and platform comparisons |
choose-host/recommended-hosts.md |
List of vetted VPS and PaaS providers with pricing and features |
choose-host/unrecommended-hosts.md |
Documented problematic hosting providers with specific issues |
choose-host/choose-host1.md |
Placeholder for future additional considerations content |
config-references/config-vars.md |
Comprehensive config variables reference with categorization and detailed descriptions |
config-references/color-names.md |
Color reference documentation moved from root (new location) |
installation/community-guides.md |
Added warnings for Replit, Northflank, and Pterodactyl-based hosts |
usage-guide/threadmenu.md |
Fixed command hierarchy formatting with consistent markdown syntax |
SUMMARY.md |
Updated table of contents to reflect new structure and moved pages |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
Member
Author
|
Oh you can't use the ai suggestions or it gets rid of the review |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Stephen <48072084+StephenDaDev@users.noreply.github.com>
StephenDaDev
dismissed
RealCyGuy’s stale review
Changes implements and files have been changed.
…/modmail-docs into stephendadev-patch-1
sebkuip
previously requested changes
Nov 19, 2025
Member
sebkuip
left a comment
sebkuip
left a comment
There was a problem hiding this comment.
I have a few things I'd like to see changes. See my list below on the exact details.
lorenzo132
approved these changes
Nov 19, 2025
StephenDaDev
dismissed
sebkuip’s stale review
lorenzo132 approved these changes on behalf of modmail-dev/wiki-team 1 hour ago
Contributor
|
There are change request left out is that intented? |
Member
Author
|
I'm not seeing any that were not addressed? |
Contributor
|
Hmm weird 
|
Contributor
|
You dont have them in the pr here? |
Member
Author
|
OOPS!! I missed that one completely... (referring to the AI comment above) |
Contributor
|
Implemented one of the other change request at #54 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.