This repository has been archived by the owner on Nov 20, 2020. It is now read-only.
Add fetch more and network status to useQuery #25
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Addresses missing bindings for
fetchMore
in #1 .What this PR does:
reason-apollo
with some pointers to the originalTS
definitions to make easier updates in the future),networkStatus
that can be used to track whetherfetchMore
is in progress,the original definition is located in
apollo-client
,swapi
example fromreason-apollo
with usage of thequery
hook usingfetchMore
andnetworkStatus
.Does it make sense to include links to the original
TS
definition fromapollo
packages? From my experience it might help other people figure out how the bindings work, do troubleshooting or update the bindings in case the original definition changes. But, it clutters the code a bit.Appreciate any advice on this and/or a code review 馃檪