-
Notifications
You must be signed in to change notification settings - Fork 52
DOCSP-32548: Breaking changes consolidation #777
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
DOCSP-32548: Breaking changes consolidation #777
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.
Requesting changes so I can see the changes after you add the links back to the breaking changes guide!
- The ``includeResultMetadata`` option for compound operation methods | ||
is ``false`` by default. See the :ref:`Built-in | ||
Methods <node-compound-operations-builtin>` section of the Compound | ||
Operations guide for more information. |
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.
S: for certain entries, a page is linked in this note, but not in the breaking changes guide. Could you go through and make sure that additional information is linked in the breaking changes guide wherever they have been linked in these admonitions?
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 catching this! I've added the links to installing the socks package (node-connect-socks) and information about compound operations (node-compound-operations-builtin).
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!
* DOCSP-32548: Breaking changes consolidation * changing admonition * deleting repeated info * review feedback (cherry picked from commit 2e0cac9)
* DOCSP-32548: Breaking changes consolidation * changing admonition * deleting repeated info * review feedback
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-32548
Staging - https://docs-mongodbcom-staging.corp.mongodb.com/node/docsworker-xlarge/DOCSP-32548-consolidate-breaking/whats-new/
Self-Review Checklist