fix: default for isLoading form state #10095
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Mentioned here #9934.
This pull request fixes an issue with the
isLoading
functionality. Currently, the default value fordefaultValues
is always set totrue
, even if it's a static value. I have a component abstraction that subscribes toisLoading
to display a loading state, but the default value was causing issues with the component's behaviour.To address this, I've updated the
defaultValues
default totrue
only if it's a function, which allows the component to support both async and staticdefaultValues
without any unexpected behaviour.