-
Notifications
You must be signed in to change notification settings - Fork 18
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
Docs: Add role-based permissions tutorial #894
Docs: Add role-based permissions tutorial #894
Conversation
DX: Assertion Testing✅ DiffThe contribution is well-structured with a clear heading, introduction, step-by-step tutorial, and wrapping up section that explains the context and content of changes. Code snippets are properly formatted, and appropriate titles provide clarity. The prerequisites section sets a good foundation for the reader before diving into the tutorial. The examples provided are easy to understand and follow. The use of comments in the code to clarify the purpose of code sections is helpful. ✅ IntegratedThe PR integrates smoothly with the rest of the documentation. It fits into the ordered tutorials, adding a valuable piece on role-based permissions, specifically on restricting command execution. The documentation structure benefits from this addition as it continues the logical sequence of tutorials and improves the depth of the authorization topic. The update to the index.mdx file ensures that the new tutorial is appropriately referenced within the overall structure, maintaining ease of navigation for the user. |
@robertjdominguez Thanks for your PR! I've assigned @robertjdominguez to review it. |
There were no recipes under the `Recipes` heading...
b7ce315
to
37aab6f
Compare
Tweaks the language a bit after re-reading it with fresh eyes.
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.
Gorgeous.
…nships-in-command-permissions
Description 📝
Adds a role-based permissions tutorial after questions from users in Discord came through. This illustrates how to limit command execution based on a relationship (e.g., an
author
can only update their own posts).Quick Links 🚀
New tutorial
Assertion Tests 🤖
A user should be able to add role-based permissions to limit command execution in their API.