Skip to content

Conversation

Zxilly
Copy link
Contributor

@Zxilly Zxilly commented Sep 7, 2021

resolve: #50

I'm not a sqlalchemy expert, and this patch works as a trick.
Please confirm if this resolve your problem. @derekn

Signed-off-by: Zxilly zhouxinyu1001@gmail.com

Signed-off-by: Zxilly <zhouxinyu1001@gmail.com>
Signed-off-by: Zxilly <zhouxinyu1001@gmail.com>
@hsluoyz hsluoyz requested review from leeqvip and ffyuanda September 7, 2021 06:01
@hsluoyz
Copy link
Member

hsluoyz commented Sep 7, 2021

@techoner @ffyuanda plz review

Copy link
Member

@ffyuanda ffyuanda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for now, though reset the metadata seems tricky.

@Zxilly
Copy link
Contributor Author

Zxilly commented Sep 8, 2021

@hsluoyz plz merge this

@hsluoyz hsluoyz merged commit 5a00fbf into pycasbin:master Sep 8, 2021
@github-actions
Copy link

github-actions bot commented Sep 8, 2021

🎉 This PR is included in version 0.4.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

how to use custom sqlalchemy model
3 participants