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

UpdateGroupingPolicy() and UpdateNamedGroupingPolicy() API for node-casbin #324

Closed
ankitaShrma opened this issue Sep 17, 2021 · 10 comments · Fixed by #419
Closed

UpdateGroupingPolicy() and UpdateNamedGroupingPolicy() API for node-casbin #324

ankitaShrma opened this issue Sep 17, 2021 · 10 comments · Fixed by #419
Assignees
@hsluoyz @Hanaffi
Labels
enhancement New feature or request released

Comments

@ankitaShrma
Copy link

There is updatePolicy(), but I don't see any update APIs for roles and other grouping policies in node version. Is this going to be implemented?
https://casbin.org/docs/en/management-api#updatenamedgroupingpolicy
@casbin-bot
Copy link
Member

@Zxilly @Gabriel-403

@casbin-bot casbin-bot added the question Further information is requested label Sep 17, 2021
@hsluoyz hsluoyz added enhancement New feature or request and removed question Further information is requested labels Sep 18, 2021
@hsluoyz
Copy link
Member

hsluoyz commented Sep 18, 2021

@MRGUOKING plz fix labeling:

image

@Shivansh-yadav13
Copy link
Member

@hsluoyz can I work on this?

@Zxilly
Copy link
Contributor

Zxilly commented Feb 11, 2022

@Shivansh-yadav13 You can feel free to make PR

@hsluoyz
Copy link
Member

hsluoyz commented Feb 11, 2022

@Shivansh-yadav13 plz work on: #332 (I ATed you actually)

@Shivansh-yadav13
Copy link
Member

removing my previous issue so that we have one place to track them

  • UpdateGroupingPolicy
  • UpdateNamedGroupingPolicy
  • GetImplicitResourcesForUser

@Shivansh-yadav13 Shivansh-yadav13 mentioned this issue Mar 11, 2022
Closed
@Hanaffi
Copy link

Hanaffi commented Mar 12, 2022

@hsluoyz @nodece Can you please assign this to me?

@Shivansh-yadav13
Copy link
Member

@Hanaffi are you up for adding these?

@hsluoyz
Copy link
Member

hsluoyz commented May 1, 2022

@Shivansh-yadav13 you can do it

@Shivansh-yadav13 Shivansh-yadav13 mentioned this issue May 3, 2022
Closed
@imp2002 imp2002 mentioned this issue Feb 1, 2023
Merged
@github-actions
Copy link

github-actions bot commented Feb 1, 2023

🎉 This issue has been resolved in version 5.23.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
Assignees

@hsluoyz hsluoyz

@Hanaffi Hanaffi

Labels
enhancement New feature or request released
Projects
None yet
Milestone
No milestone
6 participants
@hsluoyz @ankitaShrma @Zxilly @Hanaffi @casbin-bot @Shivansh-yadav13