Replies: 1 comment 1 reply
-
I like it. Please make a PR to |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
How do you feel about renaming the new Infinite Queries
defaultPageParam
argument toinitialPageParam
?The use of the word "default" implies the value can be used as a fallback when in case of page parameters, it "stands" in line with the rest of the page parameters: it can be used by explicitly "selecting" it via the Infinite Queries methods like
fetchNextPage
orfetchPreviousPage
; it's not being used in the context of "reset to default page" type of method.Furthermore, "initialPageParam" name is inline with the Query API argument
initialData
(andinitialDataUpdatedAt
) and the way it's been logically handled by queries.Let me know what you think. I can already provide a PR since I had to dig through the source code to justify my proposal anyway 😊
Beta Was this translation helpful? Give feedback.
All reactions