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

Enable/disable get and list suffixes depending on can-i auth #184

Merged
merged 8 commits into from
Apr 14, 2024

Conversation

jinnovation
Copy link
Owner

Closes #152.

This PR uses the :inapt-if-not slot in transient-suffix to disable the get and list suffixes on kele-resource depending on whether or not the user has permission to get and list the given resource for the given context.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.70%. Comparing base (7719ca6) to head (9e97219).
Report is 29 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #184      +/-   ##
==========================================
- Coverage   73.92%   71.70%   -2.23%     
==========================================
  Files           1        1              
  Lines         629      682      +53     
==========================================
+ Hits          465      489      +24     
- Misses        164      193      +29     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

kele.el Outdated Show resolved Hide resolved
kele.el Outdated Show resolved Hide resolved
@jinnovation jinnovation merged commit 60c1a94 into main Apr 14, 2024
9 checks passed
@jinnovation jinnovation deleted the jjin/can-i-get branch April 14, 2024 20:00
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.

Kele should respect authorization for listing resources
2 participants