-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
fix(collection): Add ignoreUndefined option in beginning with findOneAnd* methods. #2147
Conversation
…Delete, findOneAndReplace methods
@daprahamian Please feedback here. |
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.
Hi @changmyeong,
I prefer the original version. executeOperation
is used by a lot more than the crud operations, so adding something in there is not really ideal.
- Dan
…ods" This reverts commit 8d66f2f.
@daprahamian Thanks to feedback :) i reverted commit. |
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
@daprahamian @mbroadst |
@changmyeong sorry about that! It is merged into |
Add ignoreUndefined option in beginning with findOneAnd* methods.
NODE-2169