Skip to content

fix(forms): implement interoperability between signal forms and reactive forms#64471

Closed
leonsenft wants to merge 4 commits intoangular:mainfrom
leonsenft:signal-form-cva-interop
Closed

fix(forms): implement interoperability between signal forms and reactive forms#64471
leonsenft wants to merge 4 commits intoangular:mainfrom
leonsenft:signal-form-cva-interop

Conversation

@leonsenft
Copy link
Copy Markdown
Contributor

Add support for interoperability between signal forms and reactive forms that commit effccff had removed.

A signal forms field can once again be bound to any element or component with a ControlValueAccessor.

@leonsenft leonsenft requested a review from kirjs October 16, 2025 16:36
@leonsenft leonsenft added target: major This PR is targeted for the next major release forms: signals labels Oct 16, 2025
@ngbot ngbot bot modified the milestone: Backlog Oct 16, 2025
@leonsenft leonsenft force-pushed the signal-form-cva-interop branch from 98fb17d to 8b08f30 Compare October 16, 2025 17:12
@pullapprove pullapprove bot requested a review from crisbeto October 16, 2025 17:12
@leonsenft leonsenft force-pushed the signal-form-cva-interop branch 2 times, most recently from 70d1930 to dc2f89e Compare October 17, 2025 15:57
…ive forms

Add support for interoperability between signal forms and reactive forms that
commit effccff had removed.

A signal forms field can once again be bound to any element or component with a
`ControlValueAccessor`.
These methods are only intended to be used internally within framework
instructions. Prefix them with `ɵ` to indicate that they are
framework-private and should not be called from user code.
…eAccessor`

This corresponds with fixes made for native and custom signal form
controls: angular#64483.
@leonsenft leonsenft force-pushed the signal-form-cva-interop branch from dc2f89e to d59442a Compare October 17, 2025 18:02
@pullapprove pullapprove bot requested a review from devversion October 17, 2025 21:17
Copy link
Copy Markdown
Contributor

@kirjs kirjs left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api

@leonsenft leonsenft added the action: merge The PR is ready for merge by the caretaker label Oct 18, 2025
thePunderWoman pushed a commit that referenced this pull request Oct 20, 2025
…#64471)

These methods are only intended to be used internally within framework
instructions. Prefix them with `ɵ` to indicate that they are
framework-private and should not be called from user code.

PR Close #64471
thePunderWoman pushed a commit that referenced this pull request Oct 20, 2025
…eAccessor` (#64471)

This corresponds with fixes made for native and custom signal form
controls: #64483.

PR Close #64471
@thePunderWoman
Copy link
Copy Markdown
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

@leonsenft leonsenft deleted the signal-form-cva-interop branch November 6, 2025 17:41
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: forms forms: signals target: major This PR is targeted for the next major release

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants