You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This hook currently returns { fetchSearchResults, isLoading }. I noticed two things:
We're using two different verbs to describe the same action: "fetch" and "load"
The action is verbose but the state is terse
I think we can reconcile these differences and create more symmetry between the action and the state. I suggest erring on the side of verbosity to avoid potential conflicts with the consumer's variables, e.g. a local isLoading variable. I'd suggest { fetchSearchResults, isFetchingSearchResults }.
The text was updated successfully, but these errors were encountered:
This hook currently returns
{ fetchSearchResults, isLoading }
. I noticed two things:I think we can reconcile these differences and create more symmetry between the action and the state. I suggest erring on the side of verbosity to avoid potential conflicts with the consumer's variables, e.g. a local
isLoading
variable. I'd suggest{ fetchSearchResults, isFetchingSearchResults }
.The text was updated successfully, but these errors were encountered: