-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat: auto field value update #729
Conversation
|
129e6c0
to
ad90606
Compare
Deploying conform with Cloudflare Pages
|
ad90606
to
5ab22b6
Compare
More templates
@conform-to/dom
@conform-to/react
@conform-to/yup
@conform-to/zod
@conform-to/validitystate
commit: |
5ab22b6
to
dbddf13
Compare
Fix #600
This is my 2nd attempt to remove the need of passing the
key
to each input manually by updating the fields value automatically. As mentioned in #728...However, the previous solution made an wrong assumption that people will always subscribe to the
initialValue
when setting up a field which conflicts with (2). As there seems to be no way to associate a field with a subscription, we will either need to re-run the same side effect on every subscription, or, have theuseForm()
hook subscribed to all key changes so it will always re-render and run the side effect in one single place.This PR uses the later approach.