Skip to content

docs: rewrite recursion and max depth page#515

Merged
miparnisari merged 3 commits intoauthzed:mainfrom
ivanauth:fix/docs-issue-384-rewrite-recursion-max-depth
Apr 8, 2026
Merged

docs: rewrite recursion and max depth page#515
miparnisari merged 3 commits intoauthzed:mainfrom
ivanauth:fix/docs-issue-384-rewrite-recursion-max-depth

Conversation

@ivanauth
Copy link
Copy Markdown
Contributor

@ivanauth ivanauth commented Feb 6, 2026

Rewrites the recursion and max depth page for clarity.

  • Rename title to 'Cyclical Relationships and Traversal Limits'
  • Add ASCII diagrams showing traversal and cycle detection
  • Clarify max depth behavior and configuration
  • Reorganize FAQ section

Fixes #384

## Common Questions

### What do I do about a max depth error on CheckPermission?
### What do I do about a max depth error?
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed, thanks

Much academic literature has been written about this topic (which we won't repeat here),
but the very question raises whether allowing such an approach is semantically valid.

As a real example, imagine the following schema and relationships:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

can we leave this example please? it is quite interesting

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

sure!

@ivanauth ivanauth force-pushed the fix/docs-issue-384-rewrite-recursion-max-depth branch from 2817821 to 591fd98 Compare March 30, 2026 14:02
@ivanauth ivanauth requested a review from miparnisari April 3, 2026 16:25
@miparnisari miparnisari enabled auto-merge (squash) April 8, 2026 17:19
@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Apr 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Apr 8, 2026 5:25pm

Request Review

@authzed authzed deleted a comment from vercel bot Apr 8, 2026
@tstirrat15 tstirrat15 force-pushed the fix/docs-issue-384-rewrite-recursion-max-depth branch from 591fd98 to b7f05ba Compare April 8, 2026 17:33
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 8, 2026

Preview deployment status for this pull request.

Name Status Preview Updated (UTC)
docs 🟢 Ready Visit Preview Apr 08, 2026 05:36pm

@miparnisari miparnisari merged commit 49b767b into authzed:main Apr 8, 2026
6 of 7 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Apr 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rewrite "recursion and max depth" page

2 participants