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

Docs: Add role-based permissions tutorial #894

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

robertjdominguez
Copy link
Collaborator

@robertjdominguez robertjdominguez commented Feb 12, 2025

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.

@hasura-bot
Copy link
Contributor

hasura-bot commented Feb 12, 2025

DX: Assertion Testing

✅ Diff

The PR provides a clear, step-by-step tutorial on how to restrict command execution using role-based permissions. It adequately explains the changes necessary at the file level with suitable examples and necessary prerequisites. The use of comments within the codeblocks helps clarify the purpose of the code snippets.

✅ Integrated

The new documentation integrates well with the existing set of tutorials. It's correctly listed in the index file, which ensures users can find it alongside related tutorials. The content contributes positively to the overall usability of the documentation by extending the tutorials with a practical example of implementing role-based command execution, successfully meeting the usability assertion.

@hasura-bot
Copy link
Contributor

@robertjdominguez Thanks for your PR! I've assigned @robertjdominguez to review it.

@robertjdominguez robertjdominguez force-pushed the robdominguez/doc-2420-add-recipe-for-relationships-in-command-permissions branch from b7ce315 to 37aab6f Compare February 12, 2025 21:56
Copy link

cloudflare-workers-and-pages bot commented Feb 12, 2025

Deploying ddn-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: d229fcb
Status: ✅  Deploy successful!
Preview URL: https://cb96684a.v3-docs-eny.pages.dev
Branch Preview URL: https://robdominguez-doc-2420-add-re.v3-docs-eny.pages.dev

View logs

Tweaks the language a bit after re-reading it with fresh eyes.
Copy link
Collaborator

@seanparkross seanparkross left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gorgeous.

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.

3 participants