-
Notifications
You must be signed in to change notification settings - Fork 21
DOCSP-44967: Data modeling in update examples #143
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-44967: Data modeling in update examples #143
Conversation
✅ Deploy Preview for docs-rust ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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 with a question ✅
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 with one optional suggestion
source/usage-examples/replace.txt
Outdated
``name`` field is ``"Landmark Coffee Shop"`` with a new document. MongoDB | ||
replaces the first document that matches the query filter. | ||
You can access the documents in the ``restaurants`` collection as instances | ||
of the BSON data type or a custom data type. To specify which data type represents |
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.
of the BSON data type or a custom data type. To specify which data type represents | |
of the ``Document`` type or a custom data type. To specify which data type represents |
Ditto elsewhere. This more closely aligns with the code example, but feel free to leave as-is if the existing language is preferred.
(cherry picked from commit 734ed06)
(cherry picked from commit 734ed06)
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-44967
Staging -
https://deploy-preview-143--docs-rust.netlify.app/usage-examples/updateOne/
https://deploy-preview-143--docs-rust.netlify.app/usage-examples/updateMany/
https://deploy-preview-143--docs-rust.netlify.app/usage-examples/replace/
Self-Review Checklist