-
Notifications
You must be signed in to change notification settings - Fork 5
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
feat: subgraph handling #78
Conversation
async function getAnnouncementsUsingSubgraph({ | ||
subgraphUrl, | ||
filter = '', | ||
pageSize = 1000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe alchemy supports page size of 10k.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But maybe it's better to just leave it at 1K, and just make a note of it. Because other subgraph providers usually only support 1k.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool updated with docs:
entity: string; | ||
lastId?: string; | ||
}): AsyncGenerator<T[], void, unknown> { | ||
const variables: { first: number; id_lt?: string } = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
More documentation in this file would really help
closing since these changes were merged in via: |
Description
Adds support for fetching announcements via a subgraph.