Skip to content
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

Fix Mongoose DeprecationWarning: collection.update is deprecated #56

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

No description provided.

@jebarjonet
Copy link

@cardimajs those should use updateMany instead of updateOne no?

@jebarjonet jebarjonet mentioned this pull request Apr 17, 2019
Closed
@coveralls
Copy link

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

Is this being pulled any time soon?

@natenrb9
Copy link

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
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@cardimajs @jebarjonet @coveralls @israelglar @natenrb9 @dsanel