githubv4 icon indicating copy to clipboard operation
githubv4 copied to clipboard
verified publisher icon shurcooL

Create a "deep-search" query execution

Open isihu opened this issue 2 years ago • 2 comments

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 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

isihu avatar Dec 06 '23 16:12 isihu

@dmitshur Sorry for ping. Is this something you'd be interested in? I'd be happy to provide a PR

isihu avatar Feb 21 '24 13:02 isihu

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.

dmitshur avatar Feb 24 '24 21:02 dmitshur