Skip to content

docs(forms): name the default error factories in the signal validators - #70824

Open
erkamyaman wants to merge 1 commit into
angular:mainfrom
erkamyaman:docs-forms-signals-validator-error-factories
Open

erkamyaman wants to merge 1 commit into
angular:mainfrom
erkamyaman:docs-forms-signals-validator-error-factories

Conversation

@erkamyaman

Copy link
Copy Markdown
Contributor

The error option of each signal forms validator said it replaces the default ValidationError.email(), ValidationError.min(minValue) and so on. ValidationError has no such members, so the generated links pointed at anchors that do not exist.

Name the factory each validator actually uses for its default error, such as emailError() and minError(), which also links to its API page.

The `error` option of each signal forms validator said it replaces the
default `ValidationError.email()`, `ValidationError.min(minValue)` and
so on. `ValidationError` has no such members, so the generated links
pointed at anchors that do not exist.

Name the factory each validator actually uses for its default error,
such as `emailError()` and `minError()`, which also links to its API
page.
@pullapprove
pullapprove Bot requested a review from JeanMeche September 19, 2026 17:14
@angular-robot angular-robot Bot added area: docs Related to the documentation area: forms labels Sep 19, 2026
@ngbot ngbot Bot added this to the Backlog milestone Sep 19, 2026
@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note labels Sep 19, 2026
@JeanMeche

Copy link
Copy Markdown
Member

caretaker note: This is a doc only change, no need to presubmit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: docs Related to the documentation area: forms merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants