1st argument changes, remove "path" and "url" options, bug fixes

[iamthesiz]

Summary

1. this removes the path and url fields in exchange for the 1st argument handling both. Auto managed useFetch doesn't append URL to provider #247
   BREAKING CHANGE 🚨

useFetch('/path')
// AND
useFetch('https://overwrite-global-url.com')

2. Fixes issue when overwriting global options, it changes for every instance of useFetch. Deleting a global option persists for all instances #250

3. fixes the pagination issue here loading bug with cache? #237 but might not fix the loading bug.

4. small bug fix with responseType

Todos

1. need more tests
   • test for useFetch('path') without the / in the path (might be fine with the current tests)
2. codesandbox for conditional auto-managed state
3. update youtube video for path to explain how the new way works
4. draft a release explaining the breaking changes
5. update all codesandboxes using path and url options. I think it's just this one and this one

[JonKrone]

Hey Alex!

Thanks for your work on this! I used this library for a toy project over the weekend and it was fairly seamless.

I saw in the issue thread that sparked this your consideration of a skip vs a null path. Can you share your reasoning for going with the path approach?

I have a slight preference for a skip option to avoid overriding the path parameter, though I understand that either option has some mental overhead for the library.

[iamthesiz]

@JonKrone I've been doing a lot of research on all the various patterns people use. It seems swr and react-query are blocking fetches with a null or falsy value. Looks like @apollo/react-hooks is using the skip option. I am open to the skip option. I wanna get some more feedback and I'll get back to you.

[iamthesiz]

@JonKrone After looking at it more, seems like @apollo/react-hooks has significantly more downloads and significantly more people using it. I might do the skip option as it might lead to easier understanding/adoption of the library. It's also more explicit. You are not the 1st person who has requested this feature.

[iamthesiz]

I wish there was a way to crowd source people's opinions on this.

[iamthesiz]

I found a way to crowdsource people's opinions on reddit. See the poll here on whether to use skip option or null value.

[iamthesiz]

Going to let more people vote before adding conditional auto-managed state. Gonna get these fixes in now though.