Skip to content

Conversation

@AliRoshaniDev
Copy link
Contributor

@AliRoshaniDev AliRoshaniDev commented Mar 31, 2021

This code sample has two problems:
"oldPagesArray?.pages" only contains an array of different page arrays, so we should not write "page.data" to filter because "page.data" is undefined and we should write "page" instead.

Another mistake is the value pushed to the "newPagesArray" and the following value must be replaced: "newData"

After using this code sample, I realized that this code has two problems:
"oldPagesArray?.pages" only contains an array of different page arrays, so we should not write "page.data" to filter because "page.data" is undefined and we should write "page" instead.
Another mistake is the value pushed to the "newPagesArray" and the following value must be replaced: "newData"
@vercel
Copy link

vercel bot commented Mar 31, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/tannerlinsley/react-query/GCM2uCicabWLFqdrK4yXj4GTsNpT
✅ Preview: https://react-query-git-fork-alimoallem27-master-tannerlinsley.vercel.app

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit f6bc8ce:

Sandbox Source
tannerlinsley/react-query: basic Configuration
tannerlinsley/react-query: basic-typescript Configuration

@AliRoshaniDev AliRoshaniDev changed the title Correction of two mistakes in the code sample docs(useInfiniteQuery): Correction of two mistakes in the code sample Apr 5, 2021
@TkDodo TkDodo merged commit 6ee897c into TanStack:master Apr 10, 2021
@tannerlinsley
Copy link
Member

🎉 This PR is included in version 3.13.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants