Skip to content

Add a note clarifying remote relationships across subgraphs are not supported in permission filters #973

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

Merged
merged 2 commits into from
Mar 27, 2025

Conversation

rakeshkky
Copy link
Member

Description 📝

This PR updates the documentation to clarify that remote relationships across subgraphs are not supported permission filters.

Quick Links 🚀

Assertion Tests 🤖

@hasura-bot
Copy link
Contributor

hasura-bot commented Mar 27, 2025

DX: Assertion Testing

✅ Diff

The addition of the info box regarding the lack of support for remote relationships in permission filters is a useful piece of information. It clearly outlines a limitation that users should be aware of when defining model permissions. The formatting using the 'info' box draws attention appropriately, and the concise explanation is easy to understand.

✅ Integrated

The added information integrates well within the context of the existing documentation. It is placed under the section where filter expressions are discussed, which is the most logical place for users to learn about potential limitations regarding remote relationships. This addition contributes positively to the overall usability by clearly setting expectations and reducing potential confusion for users developing with these permissions.

@hasura-bot
Copy link
Contributor

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

Copy link

cloudflare-workers-and-pages bot commented Mar 27, 2025

Deploying ddn-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0a47cc2
Status:⚡️  Build in progress...

View logs

@robertjdominguez robertjdominguez enabled auto-merge (squash) March 27, 2025 15:29
@robertjdominguez robertjdominguez merged commit f491522 into main Mar 27, 2025
3 of 4 checks passed
@robertjdominguez robertjdominguez deleted the rakeshkky/cross-subgraph-permission-limitation branch March 27, 2025 15:32
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