Skip to content

Fix Mongoose DeprecationWarning: collection.update is deprecated #56

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Fix Mongoose DeprecationWarning: collection.update is deprecated #56

wants to merge 2 commits into from

Conversation

@cardimajs
Copy link

@cardimajs cardimajs commented Feb 19, 2019

No description provided.

@jebarjonet
Copy link

jebarjonet commented Apr 17, 2019

@cardimajs those should use updateMany instead of updateOne no?
@jebarjonet jebarjonet mentioned this pull request Apr 17, 2019
Closed
@coveralls
Copy link

coveralls commented May 1, 2019

Coverage Status

Coverage remained the same at 100.0% when pulling df9f5c3 on cardimajs:master into 65aad31 on dsanel:master.
@cardimajs
Copy link
Author

cardimajs commented May 1, 2019 •
edited
Loading

@jebarjonet yes thanks.
@israelglar
Copy link

israelglar commented May 13, 2019

Is this being pulled any time soon?
@natenrb9
Copy link

natenrb9 commented Jun 17, 2019

Please pull this commit and update. Thanks.
dsanel added a commit that referenced this pull request Sep 3, 2019
@dsanel
Handling #50, #56, #52. Adding additional tests
530050d 
#52 Counter mongo index bug with $ne and count() #50 do not use $ne operator #56 Fix Mongoose DeprecationWarning: collection.update is deprecated
@dsanel
Copy link
Owner

dsanel commented Sep 3, 2019

https://github.com/dsanel/mongoose-delete/blob/master/CHANGELOG.md#v051

Thanks for help @cardimajs @jebarjonet
@dsanel dsanel closed this Sep 3, 2019
@nsine nsine mentioned this pull request Apr 28, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

6 participants

@cardimajs @jebarjonet @coveralls @israelglar @natenrb9 @dsanel