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.

Sign up for GitHub

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

Jump to bottom

Update docs on usePaginationFragement & transitions in Relay tutorial. #4842

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update docs on usePaginationFragement & transitions in Relay tutorial. #4842

wants to merge 1 commit into from

Commits on Nov 7, 2024

  1. Update connections-pagination.md 

    I found out that `useTransition` doesn't work when using `usePaginationFragement`. To get it to work the way it's intended in the tutorial I had to use `isLoadingNext` boolean prop that is returned from `usePaginationFragement` instead. Not sure if this a bug, or by design.

As a beginner, I spent quite some time trying to figure out why my code wasn't working, after a bit of digging I ran into some potenially related issues:
- facebook#4531 (comment)
- facebook#4526
- facebook#3082

In this PR I've updated the "Improving the Loading Experience with useTransition" example not to confuse newcomers.
    @danielstocks
    danielstocks authored Nov 7, 2024
    Configuration menu
    Copy the full SHA
    b79609b View commit details
    Browse the repository at this point in the history