studentofjs / query-plus Goto Github PK

Add tests for useFetch hook. react-hook-testing library or maybe just react-testing-library. Unsure on mocking indexedDB and a web worker, possibly a step too far

Ideas for improvements:

Use coordinates instead of mouse enter /mouse leave. This will allow for a larger area and we could do it by data attr instead of having to use a component.

• Use coordinates plus/minus x amount as the countdown zone
• Ensure there is only one countdown running at a time
• Convert setTimeout to promise

Create a component that:

• - takes the same props as usePreFetch + children + a time to execute
• - on mouseenter or focus starts timer
• - on mouseleave or blur stops timer
• - initiates prefetch when the timer exceeds time to execute
• - manages timers carefully

Update READ.ME with instructions and examples for the usePreFetch hook.

We are currently calling the api, regardless, to get the data to diff it against what is saved. If the endpoint always returns the same data, or updates on a semi regular basis (i.e. monthly) then it would be desirable to set the maxAge and not call the api if it hasn't expired. However, we will need a way to differentiate between this and endpoints where it would be desireable to check the api for new data on every call.
There appears to be two obvious solutions:

1. add a variable to the request
2. create a seperate function for fetching data, that only calls the api if none exists

I feel the first option is easier to use, but more prone to mistakes.

Issues with deployment not including dependencies.
Use version 0.6.0
Build and deploy later

• limited to 50MB in Safari
• clears after 2 weeks in Safari
• issues on iphone???

Are there any performance benefits to using this as a first layer of cache, before the space runs out?