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

feat: multiple request, policy, effect, matcher type support #181

Merged
merged 1 commit into from
Aug 17, 2021

Conversation

ffyuanda
Copy link
Member

Fix: #180
updated to match Casbin version v2.35.0
Signed-off-by: ffyuanda 46557895+ffyuanda@users.noreply.github.com

@ffyuanda ffyuanda force-pushed the v2.35.0 branch 3 times, most recently from f79c0cd to 9d0d745 Compare August 16, 2021 12:53
@ffyuanda
Copy link
Member Author

@hsluoyz @techoner plz review

@hsluoyz hsluoyz requested review from leeqvip and Zxilly August 16, 2021 13:21
@hsluoyz
Copy link
Member

hsluoyz commented Aug 16, 2021

@aprey10 @Zxilly plz review

Copy link

@aprey10 aprey10 left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

casbin/core_enforcer.py Outdated Show resolved Hide resolved
casbin/core_enforcer.py Outdated Show resolved Hide resolved
Copy link
Contributor

@Zxilly Zxilly left a comment

Choose a reason for hiding this comment

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

lgtm

Signed-off-by: ffyuanda <46557895+ffyuanda@users.noreply.github.com>
@leeqvip leeqvip merged commit 468503b into casbin:master Aug 17, 2021
github-actions bot pushed a commit that referenced this pull request Aug 17, 2021
# [1.5.0](v1.4.0...v1.5.0) (2021-08-17)

### Features

* multiple request, policy, effect, matcher type support ([#181](#181)) ([468503b](468503b))
@github-actions
Copy link

🎉 This PR is included in version 1.5.0 🎉

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.

Multiple request, policy, effect, matcher type support
5 participants