Skip to content

Updated adapter to use latest stable version of Casbin.NET (2.16.0) a…#82

Closed
stewartmcguire wants to merge 2 commits into
apache:masterfrom
stewartmcguire:value1314fix
Closed

Updated adapter to use latest stable version of Casbin.NET (2.16.0) a…#82
stewartmcguire wants to merge 2 commits into
apache:masterfrom
stewartmcguire:value1314fix

Conversation

@stewartmcguire

Copy link
Copy Markdown

…nd updated Configuration to ignore additional Value13 and Value14 columns.

…nd updated Configuration to ignore additional Value13 and Value14 columns.
@hsluoyz

hsluoyz commented Oct 3, 2025

Copy link
Copy Markdown
Member

@sagilio please review

@hsluoyz hsluoyz requested a review from sagilio October 3, 2025 12:58
@CLAassistant

CLAassistant commented Oct 3, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@hsluoyz

hsluoyz commented Oct 3, 2025

Copy link
Copy Markdown
Member

@stewartmcguire fix CI errors:

image

builder.Ignore(p => p.Value10);
builder.Ignore(p => p.Value11);
builder.Ignore(p => p.Value12);
builder.Ignore(p => p.Value13);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

These properties need be added to PersistPolicy and IPersistPolicy at Casbin.NET

@stewartmcguire stewartmcguire Oct 3, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

They already exist in the 2.16.0 release of Casbin.NET which I indicated I updated the Casbin.NET library reference to.

Comment thread Casbin.Persist.Adapter.EFCore.UnitTest/SpecialPolicyTest.cs Outdated
@hsluoyz

hsluoyz commented Oct 12, 2025

Copy link
Copy Markdown
Member

@stewartmcguire

@hsluoyz

hsluoyz commented Oct 30, 2025

Copy link
Copy Markdown
Member

Replaced by: #86

@hsluoyz hsluoyz closed this Oct 30, 2025
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.

4 participants