Pagination Customization Docs Confusing
jamesarosen opened this issue · 1 comments
jamesarosen commented
The current pagination docs are a bit confusing. They suggest
JsonApiClient::Paginating::Paginator.page_param = "page[number]"
JsonApiClient::Paginating::Paginator.per_page_param = "page[size]"But JsonApiClient::Query::Builder#pagination_params wraps those in a page: {…}, resulting in URLs like
?page[page[number]]=2&page[page[size]]=20
Without overriding #pagination_params, there's no way to get rid of the page[…] wrapping. Moving the wrapping to page_param and per_page_param probably requires a major version bump. In the meantime, a little more documentation would help.
senid231 commented
Readme fixed.
Feel free to comment here if needed.