You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
consider the query of the pagination example in your README.
How about providing a second function that, instead of returning the query as is, would deep-query the result and follow any pageInfo object until HasNextPage is false? The new results can just be appended to the existing one so the return type of the 2 functions wouldn't differ.
The API for it could be
err=ghClient.DeepQuery(ctx, &query, nil)
Do you see anything blocking this? We will need to implement this in the future for our specific use case anyway, might as well make it general and provide a PR here.
Kind regards
The text was updated successfully, but these errors were encountered:
This sounds interesting, can you say more about how it would work? Would it only handle one level of pagination or are you thinking about multiple nested levels of pagination too?
I don't know when I'll have time to review a PR for functionality like this, but if you don't mind that feel free to send one, or just a link to where it's already done. Thanks.
Hi there,
consider the query of the pagination example in your README.
How about providing a second function that, instead of returning the query as is, would deep-query the result and follow any
pageInfo
object untilHasNextPage
is false? The new results can just be appended to the existing one so the return type of the 2 functions wouldn't differ.The API for it could be
Do you see anything blocking this? We will need to implement this in the future for our specific use case anyway, might as well make it general and provide a PR here.
Kind regards
The text was updated successfully, but these errors were encountered: