-
Notifications
You must be signed in to change notification settings - Fork 34
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
Update documentation #3687
Update documentation #3687
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.
LGTM
@annastuchlik could you take a look at this PR? |
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 only added one minor suggestion.
Another one is to add the note in a separate file in the _common
folder and add it in appropriate places using the include
directive, for example:
.. include:: _common/operator-reference.rst
This update is recommended but not critical.
4993baf
to
0281e3a
Compare
The problem with using include for linking to operator docs is that there are separate pages for repair/backup and for restore. So for now I will keep the links directly in the code, so that they don't get confused, but it can be changed later on. |
This PR fixes a bunch of documentation related issues.
Fixes #3684
Fixes #3681
Fixes #3661
Fixes #3663