Open bimusiek opened 10 months ago
@bimusiek Thank you for the suggestion. As you can guess, we didn't design @realm/react
with react-hooks/exhaustive-deps
fresh in our memory :smile:
Maybe alternative is to fork ...
Maintaining additional code isn't such a good idea, and we should avoid it.
As @realm/react
is still early in development, I suppose the proposed change is acceptable.
Ah, good catch. This would unfortunately be a breaking change, so we'll have to consider how we approach this.
unfortunately be a breaking change
Imho, the major version is 0, so we can allow to introduce breaking changes
Problem
Currently there is no way to know if you forgot
deps
besides testing. We would love to usereact-hooks/exhaustive-deps
to recognise the missing deps in theuseQuery
function.Solution
If we move the
type
argument as the last one, so example usage of useQuery:Then
react-hooks/exhaustive-deps
correctly recognises missing deps (as it expects deps to be on second position and callback to be on the first one like in useEffect).Alternatives
No alternatives really. For our code base we copy-pasted
useQuery
source code and modified it to suit our needs, but I think it would be really beneficial to all developers using @realm/react.Maybe alternative is to fork
react-hooks/exhaustive-deps
and make it work forrealm-hooks
:)How important is this improvement for you?
Fairly niche but nice to have anyway
Feature would mainly be used with
Atlas Device Sync