-
Notifications
You must be signed in to change notification settings - Fork 9
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
New docs #423
Conversation
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.
@sneridagh Don't most of these new how-to docs belong in volto's docs rather than in volto-light-theme?
@davisagli I've amended the docs, so this PR should go first before #420 . Ready for your review! |
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.
@sneridagh Basically looks good to me.
…d active voices, as well as "we" and "you", but I don't have the energy to go through it all. There's also some marketing stuff that I don't want to touch.
This reverts commit ad60100.
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.
I've completed my review by pushing commits. I left a few comments for consideration. Otherwise it is good to merge, and we can check RTD to verify it works.
@stevepiercy thanks!! |
Let's verify that the home page URL is correct. https://volto-light-theme.readthedocs.io/ https://volto-light-theme.readthedocs.io/sitemap.xml https://volto-light-theme.readthedocs.io/robots.txt Oopsie, one mistake. PR incoming. No changelog needed. |
No description provided.