Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(#17109): sync p-inputnumber with formcontrol patchvalue #17347

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ANANTH4004
Copy link

This PR addresses issue #17109, which reports a bug with p-inputNumber not syncing correctly with FormControl when using the patchValue method.

Root Cause
The p-inputNumber component was not updating its internal value when the patchValue method was used on the associated FormControl. This issue caused the UI to remain out of sync with the form control.

Fix
Updated the p-inputNumber component to listen to FormControl value changes and properly update its state whenever patchValue is called.
Testing
Manually tested the fix with a reactive form setup.
Verified that calling patchValue correctly updates the value displayed by p-inputNumber.
Closes #17109.

@mertsincan mertsincan added the Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated. label Jan 15, 2025
@mertsincan
Copy link
Member

Could you please resolve conflicts in PR? Then, we'll review it. Thanks a lot!

Copy link

vercel bot commented Jan 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Updated (UTC)
primeng ⬜️ Ignored (Inspect) Visit Preview Jan 16, 2025 1:00pm
primeng-v18 ⬜️ Ignored (Inspect) Visit Preview Jan 16, 2025 1:00pm

Copy link

vercel bot commented Jan 16, 2025

@ANANTH4004 is attempting to deploy a commit to the primetek Team on Vercel.

A member of the Team first needs to authorize it.

@ANANTH4004
Copy link
Author

Conflicts are resolved Please review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Needs Revision The pull request can't be merged. Conflicts need to be corrected or documentation and code updated.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Component: p-inputNumber
2 participants