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.

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

Marshmallow v3 Compatibility #11

Merged
merged 7 commits into from
Apr 7, 2021
Merged

Conversation

qcho
Copy link

@qcho qcho commented Oct 2, 2019

Wasn't working with marshmallow v3 because of several incompatible changes.

I managed to make it work with these changes.

The func(v) call doesn't perform validations any-more. So I removed the wrong "errors" second param. But probably you need to advice where to perform this validation from now on if necessary.

@qcho qcho changed the title Marshmallow v3 Marshmallow v3 Compatibility Oct 2, 2019
@coveralls
Copy link

coveralls commented Oct 3, 2019

Coverage Status

Coverage increased (+0.4%) to 95.178% when pulling 8ecace4 on qcho:marshmallow-v3 into a168eb5 on MongoEngine:master.

@qcho
Copy link
Author

qcho commented Oct 4, 2019

python 2 is broken with this change.
Also toasted doesn't work with Marshmallow v3 yet (lyft/toasted-marshmallow#14)

Qcho added 2 commits October 4, 2019 05:16
…w v3 compatibility.

If you need python2 and/or toasted support stick with v0.10.0
@qcho
Copy link
Author

qcho commented Oct 4, 2019

Bumped version to 0.30.0 (30 for v3) because this would include breaking changes.

  1. marshmallow v3 needs >= p35.
  2. toasted doesn't support mm-v3 yet.

Tests outside these are passing.

People can stick with v0.10.0 if that's a problem.

@nachimehta
Copy link

@qcho I am getting a problem when trying to use your branch (and the main branch too btw) to execute the tutorial code.

I get this error TypeError: _make_object() got an unexpected keyword argument 'many'

Any ideas?

@rwpino
Copy link

rwpino commented Nov 26, 2020

any progress on this guys ?

@kaggrwal
Copy link

kaggrwal commented Dec 2, 2020

Any progress ??

@tschaume
Copy link
Collaborator

tschaume commented Apr 6, 2021

@rwpino @kaggrwal also see #13

@tschaume tschaume merged commit 9fa8cc1 into MongoEngine:master Apr 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants