★ wanayoo — archive 1999 https://github.com/realpython/materials/pull/111Nouvelle recherche | Portail wanayoo
Skip to content

/ materials

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

FIX: ma.ModelSchema removed in Flask-Marshmallow 0.12.0 #111

Open
wants to merge 4 commits into
base: master
from
Open

FIX: ma.ModelSchema removed in Flask-Marshmallow 0.12.0 #111

wants to merge 4 commits into from

Conversation

@Erff
Copy link

Erff commented May 2, 2020

Replaced ma.ModelSchema by ma.SQLAlchemyAutoSchema
https://github.com/marshmallow-code/flask-marshmallow/blob/dev/CHANGELOG.rst#0120-2020-04-26
Erff and others added 2 commits May 2, 2020
@Erff
FIX: ma.ModelSchema removed
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
3d0f4ce 
https://github.com/marshmallow-code/flask-marshmallow/blob/dev/CHANGELOG.rst#0120-2020-04-26
jpeldoza
FIX: Failed tests
Loading status checks…
69e6a84
@jackchi
jackchi reviewed May 13, 2020
View changes
flask-connexion-rest-part-2/version_1/models.py Outdated Show resolved Hide resolved
jpeldoza and others added 2 commits May 13, 2020
jpeldoza
removing sqla_session no longer needed.
Loading status checks…
254edb2
@Erff
Merge branch 'master' into master
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
Loading status checks…
7299685
@SouthernYoda
Copy link

SouthernYoda commented May 17, 2020

Thanks for posting a fix!
@Erff Erff closed this May 17, 2020
@Erff Erff reopened this May 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jackchi jackchi

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@Erff @SouthernYoda @jackchi
You can’t perform that action at this time.