-
-
Notifications
You must be signed in to change notification settings - Fork 585
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
docs: improve additional-hostnames.md #5045
Conversation
Removed confusing string quotes that doesn't work in the config and example of console command that adds the hostnames for you.
I don't know why quote marks around the hostname entries wouldn't work... that's standard yaml. Please check again. |
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.
As said before by Randy, quotes should not be a problem but indention is wrong in the example.
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.
Added my suggestions. The additional example with ddev config ...
looks good, thanks!
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.
Sorry for the wait! Added a suggestion to help with the flow and connect examples.
Two more things I can’t offer as suggestions, if you’re willing to update them:
-
The opening sentence should probably describe both methods before illustrating them:
You can add hostnames to a project by editing its config file or using the
ddev config
command.Use the
additional_hostnames
array in.ddev/config.yaml
: -
Add
yaml
to the config snippets for syntax highlighting. (```yaml
)
Co-authored-by: Matt Stein <m@ttste.in>
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!
Removed confusing string quotes that doesn't work in the config and example of console command that adds the hostnames for you.
The Issue
Quotes around the hostnames didn't work for me.
How This PR Solves The Issue
Removes the quotes from the text and suggests a console command
Manual Testing Instructions
Automated Testing Overview
Related Issue Link(s)
Release/Deployment Notes