pycasbin/sqlalchemy-adapter

Implement update_filtered_policies()

leeqvip opened this issue · 7 comments

ffyuanda commented

Will do in this week.

Shivansh-yadav13 commented

@leeqvip can I contribute to this?

hsluoyz commented

Sure

  • 👍1
Shivansh-yadav13 commented

Hello, I wanted to ask that sec is not used anywhere inside the update_policy() should it be removed?

sqlalchemy-adapter/casbin_sqlalchemy_adapter/adapter.py

Line 194 in 2222d9a

self, sec: str, ptype: str, old_rule: [str], new_rule: [str]

Shivansh-yadav13 commented

@hsluoyz we would also have to add update_filtered_policies in the management API pycasbin

leeqvip commented

@hsluoyz we would also have to add update_filtered_policies in the management API pycasbin

Yes, waiting for your PR.

Shivansh-yadav13 commented

@leeqvip are we not going to add update_filtered_policies method here?