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

feat: use transaction in SavePolicy() #208

Merged
merged 3 commits into from
Apr 14, 2023
Merged

feat: use transaction in SavePolicy() #208

merged 3 commits into from
Apr 14, 2023

Conversation

weloe
Copy link
Contributor

@weloe weloe commented Apr 12, 2023 •
edited by hsluoyz
Loading

Fix: #207

@casbin-bot
Copy link
Member

@tangyang9464 @JalinWang @imp2002 please review

@CLAassistant
Copy link

CLAassistant commented Apr 12, 2023 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@weloe weloe mentioned this pull request Apr 12, 2023
Closed
@hsluoyz
Copy link
Member

hsluoyz commented Apr 12, 2023

@weloe fix CI:

image

@weloe weloe mentioned this pull request Apr 13, 2023
Merged
@hsluoyz hsluoyz changed the title fix: use transaction to SavePolicy feat: use transaction in SavePolicy() Apr 14, 2023
@hsluoyz hsluoyz merged commit 089b807 into casbin:master Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@imp2002 imp2002 Awaiting requested review from imp2002

@tangyang9464 tangyang9464 Awaiting requested review from tangyang9464

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

why not use Transaction to SavePolicy
4 participants
@weloe @casbin-bot @CLAassistant @hsluoyz