Skip to content

fix: broken anchor link#301

Merged
jeff-matthews merged 1 commit into
mainfrom
fix-constraints-link
May 29, 2026
Merged

fix: broken anchor link#301
jeff-matthews merged 1 commit into
mainfrom
fix-constraints-link

Conversation

@jeff-matthews
Copy link
Copy Markdown
Contributor

@jeff-matthews jeff-matthews commented May 29, 2026

Summary by CodeRabbit

  • Documentation
    • Updated developer documentation links in the nodes guide to direct users to Property rules for property configuration guidance.

Review Change Stack

@jeff-matthews jeff-matthews self-assigned this May 29, 2026
@jeff-matthews jeff-matthews added the bug Something isn't working label May 29, 2026
@jeff-matthews jeff-matthews requested a review from martinsohn May 29, 2026 14:07
@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 29, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
bloodhound 🟢 Ready View Preview May 29, 2026, 2:11 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Copy link
Copy Markdown
Contributor

@martinsohn martinsohn left a comment

Choose a reason for hiding this comment

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

Looks good!

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1ce90225-b51b-4bd4-8d31-b949e5e5af23

📥 Commits

Reviewing files that changed from the base of the PR and between 7dd3f36 and f599563.

📒 Files selected for processing (1)
  • docs/opengraph/developer/nodes.mdx

Walkthrough

This PR updates a single internal documentation reference link in the nodes developer guide. The "properties" section guidance link is redirected from Constraints to Property rules, improving navigation within the developer documentation.

Changes

Documentation Reference Links

Layer / File(s) Summary
Properties guidance documentation link
docs/opengraph/developer/nodes.mdx
The properties section description link is updated to point to the Property rules section instead of Constraints, while keeping the Reserved properties reference intact.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A link was lost in the morning dew,
From Constraints old to Rules brand new,
The properties guide now points the way,
To better docs—hip-hop hooray! 📚✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: fixing a broken anchor link from 'Constraints' to 'Property rules' in the documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-constraints-link

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jeff-matthews jeff-matthews merged commit 899aa90 into main May 29, 2026
3 checks passed
@jeff-matthews jeff-matthews deleted the fix-constraints-link branch May 29, 2026 14:15
@github-actions github-actions Bot locked and limited conversation to collaborators May 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants