Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add paging to query.py #5

Open
MichaelCurrin opened this issue Dec 19, 2019 · 0 comments
Open

Add paging to query.py #5

MichaelCurrin opened this issue Dec 19, 2019 · 0 comments
Labels
enhancement New feature or request

Comments

@MichaelCurrin
Copy link
Owner

MichaelCurrin commented Dec 19, 2019

Also update usage doc.

See code such as

repo_page_info = repositories['pageInfo']
        if repo_page_info['hasNextPage']:
            variables['cursor'] = repo_page_info['endCursor']
        else:
            break

Test against queries with and without pageInfo in their output.
If cursor/after is provided in variables (or some flag to prevent paging esp on first page), then do not do automatic paging.

Consider how the output looks (printing data as you go or joining data in a larger file and then writing).

Also see if this is even needed. The paging is handled specifically in other Python scripts, where the case is not general.

@MichaelCurrin MichaelCurrin added the enhancement New feature or request label Dec 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant