Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(pagination): offset type casting #2964

Merged
merged 3 commits into from
Nov 8, 2024

Conversation

bodinsamuel
Copy link
Collaborator

Describe your changes

  • Do not cast offset as string
    Not sure why it was done this way. I assume that those values were set to URL and supposedly be string only (Add nango.paginate helper for sync/action scripts #1103) but at the same time, it's set as record<string, any directly.
    To be cautious I have introduced an if.

@bodinsamuel bodinsamuel self-assigned this Nov 8, 2024
@bodinsamuel bodinsamuel enabled auto-merge (squash) November 8, 2024 11:11
@bodinsamuel bodinsamuel merged commit 029e837 into master Nov 8, 2024
20 checks passed
@bodinsamuel bodinsamuel deleted the sam/24_11_08/fix/offset-casting branch November 8, 2024 11:20
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants