-
Notifications
You must be signed in to change notification settings - Fork 4
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