search

kurierjs / kurier

TypeScript framework to create JSON:API compliant APIs
https://kurier.readthedocs.io/en/latest/
MIT License
61 stars 9 forks source link

fix: type of params.page.* #367

Closed nelson6e65 closed 9 months ago

nelson6e65 commented 10 months ago

Those values are actually strings.

?page[limit]=19&page[number]=19&page[after:id]=SayLOC5U5pMj2KQlxCo0

{page: { limit: '19', number: '19', 'after:id': 'SayLOC5U5pMj2KQlxCo0' }}
joelalejandro commented 9 months ago

@nelson6e65 Unfortunately I need to revert this because KnexProcessor uses this to do math around the paging. It's weird that the CI check didn't catch it :/