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

[bug] Limit the number of nodes passed to graphql #35

Merged
merged 1 commit into from
May 27, 2024

Conversation

gerardsegarra
Copy link
Contributor

Graphql won't accept more than 100 jobs. Let's solve the bug and implement a proper solution on next PRs.

@gerardsegarra gerardsegarra requested a review from a team May 24, 2024 12:16
Copy link

@Sturgelose Sturgelose left a comment

Choose a reason for hiding this comment

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

Approved! Do we have a ticket to iterate on this bug/limitation?

Copy link
Collaborator

@arponpes arponpes left a comment

Choose a reason for hiding this comment

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

is there a ticket for doing the iteration?

Edit: I just saw Erik's comment 😅

@gerardsegarra gerardsegarra merged commit 38cbb56 into main May 27, 2024
2 checks passed
@gerardsegarra gerardsegarra deleted the limit-nodes-passed-to-graphql branch May 27, 2024 07:13
@gerardsegarra
Copy link
Contributor Author

Approved! Do we have a ticket to iterate on this bug/limitation?

No, will create it right now.

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.

None yet

3 participants