Codecov Report

Merging #1057 (60e0059) into main (4f0f216) will decrease coverage by 5.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1057      +/-   ##
==========================================
- Coverage   66.77%   61.56%   -5.22%     
==========================================
  Files         241      107     -134     
  Lines       14668     1137   -13531     
  Branches      115      115              
==========================================
- Hits         9795      700    -9095     
+ Misses       3882      412    -3470     
+ Partials      991       25     -966     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4f0f216...60e0059. Read the comment docs.

Does it conflict with this PR #1028?

Does it conflict with this PR #1028?

It does yes. We should pop in a call and discuss.

Sorry, I put the PR in an "auto-merge" mode before lunch 🤦. But please, let me know if you need any help to solve the conflicts.

I talked with @BrunoQuaresma about this and will be closing this PR.

This PR implements cursor based pagination. Offset pagination is vastly simpler and will be sufficient for the UI. The downside of offset of pagination happens with live data. I can circumvent this by adding a created_after query param to enable cursor based pagination in the forward direction. This means api scripts can use a cursor, while applications like the UI can use an offset.