Hook form: Allow to get current value from fieldname #5
Conversation
gaku-sei
changed the title
| @@ -36,6 +36,7 @@ module Form = { | |||
| clearErrors: (. string) => unit, | |||
| setError: (. string, Error.t) => unit, | |||
| reset: (~defaultValues: Js.Dict.t<Value.t>=?, unit) => unit, | |||
| getValues: (~fieldName: string) => string, | |||
There was a problem hiding this comment.
Sorry Thien, this will not always return a string, it can return any valid JS Object, so the type should be option<Js.Json.t> and we need some decoders here.
Also we could add getAllValues(~fieldNames: array<string>): array<option<Js.Json.t>> as the original function can accept an array 👍
There was a problem hiding this comment.
@gaku-sei -san. Sorry i might miss sth but I get the one when try to return type option<...>
🙇
Also you meant get another one called getAllValues side by side with getValues Kevin-san?
Actually the: getValues in react-hook-form are quite flexible
getValues(name: string | array<string>|undefined) => object
_ only string name: 1 return data for that field
_ array string field names: return object that contains all fields
_ undefined: obj all values in the form
I tried to achieve it but cannot even I just tried achieve 1 and 3 which I used to define
getValues: (~fieldName: string=?,()) => string,
but somehow when using it getValues() => it doesnt return me the obj that contain whole form
Aim to solve this one https://github.com/scoville/ats/pull/143
I also added demo in example
Screen.Recording.2021-05-22.at.10.10.58.mov