Support cursor pagination for fetching requestIdentifiers#533
Support cursor pagination for fetching requestIdentifiers#533anotherminh wants to merge 9 commits intomainfrom
Conversation
| input: { | ||
| requestIds: $requestIds | ||
| input: { requestIds: $requestIds } | ||
| filterBy: { |
There was a problem hiding this comment.
was this even working before with those values in input? 😅
|
You have used all of your free Bugbot PR reviews. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
|
Migrated to transcend-io/tools#12 |
|
Fwiw, I was working on this. I was testing out the changes, but it turned out that cursor pagination wasn't working because there was a default order applied. That meant the results were different when switching from offset to cursor pagination. See transcend-io/main#40614 |
|
Do you want me to take over the new PR as well, or let you handle it? |
|
@bencmbrook I'm realizing you might not have seen my comments above since you weren't tagged. Do you want me to take this back over from you or are you wanting to see it to completion? |
|
@iamtheluckyest I hadn't—thanks for the tag! Please do feel free to take over that PR. I'd assigned it to Mike arbitrarily. Lmk if you need any help setting up the repo, but the CONTRIBUTING.md should have you covered |
4 participants
Related Issues
Security Implications
[none]
System Availability
[none]