Check negative limit command parameter (#358) #359

Merged

6543 merged 3 commits from hamdouni/tea:fix-358 into master 5 months ago

hamdouni commented 6 months ago

fix #358

hamdouni added 1 commit 6 months ago

fc35481233 Check negative limit command parameter (#358 )

fix #358

techknowlogick approved these changes 6 months ago

noerw commented 6 months ago

@6543 should this be enforced in the sdk?

noerw approved these changes 6 months ago

noerw added the

kind/bug

label 6 months ago

noerw added this to the v0.8.0 milestone 6 months ago

6543 commented 6 months ago

Collaborator

@hamdouni can you "update" this pull :)

@noerw good question - in short, on -1 i would expect no pagination or just a fallback to default page size if enforced ...

techknowlogick added 1 commit 6 months ago

4c85ef754a Merge branch 'master' into fix-358

lunny added 1 commit 5 months ago

c98ed52213 Merge branch 'master' into fix-358

6543 merged commit ffdbdb3d02 into master 5 months ago

6543 referenced this issue from a commit 5 months ago

Check negative limit command parameter (#358) (#359)

Reviewers

techknowlogick approved these changes 6 months ago

noerw approved these changes 6 months ago

continuous-integration/drone/pr Build is passing

Details

The pull request has been merged as ffdbdb3d02.

Sign in to join this conversation.

No Assignees

5 Participants

Notifications

Due Date

No due date set.