alduino / api-utils Goto Github PK

Allowing a null request means it has to be possible to have a null return, which means we have to handle this case. Seeing as handling this case increases code complexity and you rarely, if ever, would need to use a null query (unlike swr where you might, here requests are sent when you specifically tell them to, so you can check yourself if you want to just not send it), it would make more sense to not have this as an option.

The api provider is re-exported from a library, so it should only affect that library.

Currently, the provider is just a global React context that has one instance created in api-utils. Instead, each library should create one for itself, maybe by calling a function that returns the ApiProvider component and the useSwr hooks (or it could be a parameter passed to them, or a value in the Endpoints)

There's a LICENSE.md for the MIT license but it's not specified in the package.json file.