-
Notifications
You must be signed in to change notification settings - Fork 400
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
Fixes mongoengine depreciation issue #923
Conversation
Thanks! It looks like the tests are now passing. 🎉 Do you have an understanding of the cause of that deprecation warning? Is there a change to make in |
I honestly don't know and have very little knowledge of mongo. I have a big PR that I hope you will accept [hint][wink][hint] soon, and this was a blocker. I didn't look into it further than just getting the tests to go green. |
Nice, we came to the same conclusion. #924 |
LOL, great minds think alike ;) |
PS. i used single quotes because the rest of the code uses single quotes in that file. |
Too used to Black I guess. It’s been on the table for a while #843. |
Are you merging #924 then? Do I close this one? |
Yes, please. I marked you as a co-author for #924. |
Ugh, stupid button :P |
Fixes #922
mongoengine
throwing aDeprecationWarning