Skip to content

Endpoint for fetching congress spendings as CSV #6

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

Closed
wants to merge 4 commits into from

Conversation

kacperzuk-neti
Copy link
Collaborator

@kacperzuk-neti kacperzuk-neti commented Aug 21, 2024

@kacperzuk-neti kacperzuk-neti self-assigned this Aug 21, 2024
@kacperzuk-neti kacperzuk-neti force-pushed the BLOCKCHAIN-494/congress-spendings branch from f82c869 to 953fbcd Compare September 27, 2024 13:51
@DorianSternVukotic
Copy link
Collaborator

conflicts

@kacperzuk-neti kacperzuk-neti force-pushed the BLOCKCHAIN-494/congress-spendings branch from 953fbcd to d11ffeb Compare October 1, 2024 09:52
@kacperzuk-neti kacperzuk-neti force-pushed the BLOCKCHAIN-494/congress-spendings branch from 618b370 to e927d4f Compare October 1, 2024 09:56
@kacperzuk-neti
Copy link
Collaborator Author

conflicts

Fixed

Comment on lines 92 to 93
assetTransfers(first: 50, after: $after, filter: { asset: { notEqualTo: "1" }, fromId: { equalTo: $userId } }) {
nodes {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh this only looks at first 50, we def need to change that for report generation

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its first 50 after the cursor. So 50 per page and it fetches all pages - see queryAllPages function.
Does it not fetch more than 50 in practice?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants