-
-
Notifications
You must be signed in to change notification settings - Fork 468
Rename pagination input #2724
buddhikasranasinghe
started this conversation in
Ideas
Rename pagination input
#2724
In the pagination directive, to define how many rows should be displayed per page, we need to pass a value via the 'first' parameter. This parameter name doesn't actually convey its intended meaning. I suggest that 'limit' or 'take' would be better alternatives.
type Query {
posts(
"Limits number of fetched items."
first: Int! # limit or take instead of first
"The offset from which items are returned."
page: Int
): PostPaginator
}
All reactions
Replies: 1 comment
It just reflects the GraphQL documentation, which suggests a more cursor-like approach: https://graphql.org/learn/pagination/
All reactions
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment