Skip to content

Conversation

@stewartmcguire
Copy link

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

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

hsluoyz commented Oct 3, 2025

@sagilio please review

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

CLAassistant commented Oct 3, 2025

CLA assistant check
All committers have signed the CLA.

@hsluoyz
Copy link
Member

hsluoyz commented Oct 3, 2025

@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
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

Copy link
Author

@stewartmcguire stewartmcguire Oct 3, 2025

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.

@hsluoyz
Copy link
Member

hsluoyz commented Oct 12, 2025

@stewartmcguire

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