Skip to content

docs: fix broken anchor links in low-level form guide - #7055

Merged
masenf merged 2 commits into
reflex-dev:mainfrom
Jacky-J126:docs/fix-broken-anchor-links
Sep 8, 2026
Merged

docs: fix broken anchor links in low-level form guide#7055
masenf merged 2 commits into
reflex-dev:mainfrom
Jacky-J126:docs/fix-broken-anchor-links

Conversation

@Jacky-J126

@Jacky-J126 Jacky-J126 commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

All Submissions:

  • Have you followed the guidelines stated in CONTRIBUTING.md file?
  • Have you checked to ensure there aren't any other open Pull Requests for the desired changed?

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Description

In docs/library/forms/form-ll.md, the Form Message paragraph links "client side validation" and "server side validation" to #client-side-validation and #server-side-validation, but this page has no such headings, so both anchors are dead.

This PR points both links to the existing ## Validation section instead.

Verification

  • Checked every #anchor link on the page against its headings using both GitHub's anchor rules and the reflex.dev docs slug rules: 2 broken links before the change, 0 after.
  • codespell and ruff format --check pass on the modified file.

Review in cubic

@Jacky-J126
Jacky-J126 requested review from a team and Alek99 as code owners September 7, 2026 04:47

@cubic-dev-ai cubic-dev-ai Bot left a comment

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.

No issues found across 1 file

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Updates the low-level form guide to repair its validation links and clarify how form validation works.

  • Adds dedicated client-side and server-side validation headings.
  • Corrects the force_match prop attribution from Form Control to Form Message.
  • Clarifies the roles of type, match, and as_child.

Confidence Score: 5/5

The documentation-only change appears safe to merge and leaves no actionable findings.

The added headings resolve the broken anchors, and the accompanying wording corrections accurately describe the documented validation props and behavior.

Important Files Changed

Filename Overview
docs/library/forms/form-ll.md Repairs validation anchors and improves the accuracy and organization of the form-validation documentation.

Reviews (2): Last reviewed commit: "docs: clarify low-level form validation ..." | Re-trigger Greptile

@codspeed-hq

codspeed-hq Bot commented Sep 7, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 32 untouched benchmarks
⏩ 8 skipped benchmarks1


Comparing Jacky-J126:docs/fix-broken-anchor-links (8506ada) with main (c49a85d)

Open in CodSpeed

Footnotes

  1. 8 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@masenf
masenf merged commit 6afa226 into reflex-dev:main Sep 8, 2026
33 checks passed
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