Conversation
* DOCSP-35945: read operations reorg * skip * small fixes * small fixes * fixes - RM and moved a section
jmikola
left a comment
jmikola
left a comment
There was a problem hiding this comment.
One question about inconsistent and/or redundant anchors but LGTM.
| @@ -0,0 +1,157 @@ | |||
| .. _laravel-fundamentals-search-text: | |||
| .. _laravel-retrieve-text-search: | |||
There was a problem hiding this comment.
These anchors seem inconsistent with the names in retrieve.txt:
.. _laravel-fundamentals-retrieve-documents:
.. _laravel-fundamentals-read-retrieve:
Is there even a need for multiple anchors at the same location? Presumably, the references to these could just be updated to use the same name.
There was a problem hiding this comment.
We have some guidelines around moving pages/content - https://wiki.corp.mongodb.com/spaces/DE/pages/201997355/How+To+Add+or+Rename+an+rST+Page+or+Section
That's the reason that this PR contains multiple anchors - preserving the original section headings and adding more targeted ones that match the current file tree structure. We could probably develop better guidelines around the specific words we use in these anchors though!
5 participants
Merges the read operation reorganization into 5.2 (seems like there was an issue where the 5.2 branch was created but not included in the front-ports)
Checklist