-
Notifications
You must be signed in to change notification settings - Fork 5
Open
Description
use-x/src/use-fetch/use-fetch.solution.ts
Lines 110 to 113 in ca70f15
if (error.name === "AbortError") { | |
setError(null); | |
setData(null); | |
} |
Thanks for putting this test suite together! After completing the useFetch
hook, I went back and compared my solution to the committed solution and noticed that this was the one functionality deviation between what I wrote and the expected solution. I would expect that this setError(null)
is not actually needed, as the error is set to null at the top of the hook on line 91.
I may be missing an edge case, but the test suite didn't appear to be affected by the absence of that line.
Metadata
Metadata
Assignees
Labels
No labels