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

#527 - Ensure offset is respected in JSON:API. #698

Merged
merged 2 commits into from
Feb 22, 2024

Conversation

fiasco
Copy link
Contributor

@fiasco fiasco commented Feb 22, 2024

This pull request is for: (mark with an "x")

  • examples/*
  • modules/next
  • packages/next-drupal
  • starters/basic-starter
  • starters/graphql-starter
  • Other

GitHub Issue: #527

  • I need help adding tests. (mark with an "x")
    Code changes need test coverage. If you don't know
    how to make tests, check this box to ask for help.

Describe your changes

I've had an issue where we could not build more than 1000 pages in a sitemap in nextjs because we could not paginate JSON:API beyond that point. This seems to be because of this issue.

Copy link

vercel bot commented Feb 22, 2024

@fiasco is attempting to deploy a commit to the Chapter Three Team on Vercel.

A member of the Team first needs to authorize it.

@JohnAlbin JohnAlbin merged commit 7e89119 into chapter-three:main Feb 22, 2024
10 of 11 checks passed
@fiasco
Copy link
Contributor Author

fiasco commented Feb 22, 2024

@JohnAlbin can this get backported to 1.6?

fiasco added a commit to fiasco/next-drupal that referenced this pull request Feb 23, 2024
@fiasco fiasco mentioned this pull request Feb 23, 2024
7 tasks
JohnAlbin pushed a commit that referenced this pull request Feb 23, 2024
marcorcau pushed a commit to marcorcau/next-drupal that referenced this pull request Mar 26, 2024
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