-
Notifications
You must be signed in to change notification settings - Fork 21
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
Issue #16: Enhancing the "Using Subversion" Page: Adding a 'Troubleshooting' Section #17
base: main
Are you sure you want to change the base?
Conversation
32f8648
to
db6d7c5
Compare
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 for the contribution!
I'd just like to ask you to move this new section to the end of the document (as I think the information in the "Notes" and "See also" sections is important for everyone to read before they even encounter problems).
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 for suggesting this update, @WPLake! 🙌
I agree the new section would be helpful when something unexpected occurs during deployment, and with the other suggestion of moving it to the bottom of the doc. This could also inspire other contributors to share common problems.
The changes I suggest here are from my perspective of having read lots of WP docs, but no hard feelings if you don't agree. This is a comment only, as I don't wish to block getting this update into the handbook. Thanks again!
Co-authored-by: Brian Alexander <[email protected]>
Co-authored-by: Brian Alexander <[email protected]>
@frantorres, thanks for the updates! I agree with your note above to move this to the end of the doc. Once that's done, I think this is good to approve/merge. |
Changes requested in the Enhancing the "Using Subversion" Page: Adding a 'Troubleshooting' Section.