-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add troubleshooting section #51
Add troubleshooting section #51
Conversation
Add a bunch of potential error messages in "troubleshooting.md"
Create "troubleshooting.md"
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!
This package uses the divio four-part documentation system as described here:
https://documentation.divio.com/introduction/
I think the troubleshooting stuff is part of the how-to section so let's put it there.
Co-authored-by: Claire Foster <chris42f@gmail.com>
Co-authored-by: Claire Foster <chris42f@gmail.com>
I only have my phone not, but I agree with you other two comments (merging two points and moving them to another section). If you want to, feel free to make the changes. If not, I am back with a computer in about 3 weeks. Sorry for taking so long to respong to your feedback and suggestions. They were all good 👍 |
I cloned the files locally and finally got the changes done all propper. I believe this should be good to go, so it should be ready for another review ^_^ |
Thanks! Sorry for leaving this hanging. Some of these changes were also accidentally merged as part of PR #63 . So I've taken the reviewed version from here and incorporated it manually on the main branch. So everything here should be in main now :) |
This PR adds a troubleshooting section, as mentioned in my final comment in #47. This would close #47.