-
Notifications
You must be signed in to change notification settings - Fork 33
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-43464: operations with builders #206
base: master
Are you sure you want to change the base?
DOCSP-43464: operations with builders #206
Conversation
✅ Deploy Preview for docs-php-library 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.
Looks good so far! A few suggestions
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.
Few more small things
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.
1 404, otherwise LGTM, Thanks!
documents. To learn more about updating documents, see the | ||
:ref:`php-write-update` guide. | ||
|
||
.. important:: Only $set Operator Supported in Update Documents |
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.
That's confusing.
Update supports an aggregation pipeline, in that case most of the stages can be used.
Or by default that's update operators that are not supported by the builder yet. They are not stages even if the syntax for $set
is similar.
Pull Request Info
PR Reviewing Guidelines
JIRA - https://jira.mongodb.org/browse/DOCSP-43464
Staging - https://deploy-preview-206--docs-php-library.netlify.app/builders/
Self-Review Checklist