-
Notifications
You must be signed in to change notification settings - Fork 6
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
Topic/k1ch/Introduce Admin API - GET: roles/{key}/permissions #115
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@k1ch this looks good, just one tiny change need for adding the 400 error code.
@mikelax Once this PR is merged, should we create a tag and release since this endpoint is needed for |
@mikelax I have noticed that NODE-TESTS GitHub workflow fails sometimes. Have you experienced this before? |
@k1ch yes I think we should create a new release with this new API so we can take advantage of it. |
Overview
Remainingitems:database/layer/admin-rolepermission.js
OAS update:
Your PR Checklist 🚨
❤️ Please review the guidelines for contributing to this repository. Our goal is to merge PRs fast 💨 .
Issue Reference
Relates to #106
Open Questions or Items to Callout