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: print DeprecationWarning #301

Merged
merged 2 commits into from
Nov 20, 2018
Merged

Fix: print DeprecationWarning #301

merged 2 commits into from
Nov 20, 2018

Conversation

KSH-code
Copy link
Contributor

@KSH-code KSH-code commented Sep 18, 2018 •
edited
Loading

close #297
close #300
@jdesboeufs
change remove and update to deleteOne and updateOne

@coveralls
Copy link

coveralls commented Sep 18, 2018 •
edited
Loading

Coverage Status

Coverage remained the same at 81.783% when pulling 652823a on KSH-code:fix-error into 9f86f90 on jdesboeufs:master.

@KSH-code KSH-code force-pushed the fix-error branch 2 times, most recently from 1d035d0 to 3ea762f Compare September 18, 2018 03:13
@KSH-code KSH-code mentioned this pull request Sep 19, 2018
Closed
@ryanbarwick
Copy link
Contributor

@jdesboeufs
This is a security critical bug for us, because users cannot log out. Please merge.

@ryanbarwick ryanbarwick mentioned this pull request Sep 26, 2018
Merged
@jdesboeufs jdesboeufs merged commit 652823a into jdesboeufs:master Nov 20, 2018
@KSH-code KSH-code deleted the fix-error branch July 17, 2020 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KimGenius KimGenius KimGenius approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
5 participants
@KSH-code @coveralls @ryanbarwick @KimGenius @jdesboeufs