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

feat: Add StateForm #292

Closed
wants to merge 50 commits into from

Conversation

zombieJ added 5 commits May 17, 2019
@now

This comment has been minimized.

Copy link

commented May 18, 2019

This pull request is automatically deployed with Now.
To access deployments, click Details below or on the icon next to each push.

Latest deployment for this branch: https://rc-form-git-state-form.react-component.now.sh

zombieJ added 2 commits Jun 4, 2019

@now now bot requested a deployment to staging Jun 4, 2019 Abandoned

newStore = setValue(store, namePath, value);
});

return newStore;

This comment has been minimized.

Copy link
@afc163

afc163 Jun 4, 2019

Member

感觉怪怪的,setValue 调用了 n 次。

This comment has been minimized.

Copy link
@zombieJ

zombieJ Jun 4, 2019

Author Member

以前也是这么做的,它要出一个纯粹的 Field 需要填一下对应的值。不过这里其实每次都只有一个 Field 变更,并不是真的一串值变了。

ref: https://github.com/react-component/form/blob/master/src/createBaseForm.js#L294

@zombieJ

This comment has been minimized.

Copy link
Member Author

commented Jun 10, 2019

@zombieJ zombieJ closed this Jun 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.