Skip to content

Add filter callback to KeysetPaginator and new method Compare::withValue()#153

Merged
vjik merged 14 commits intomasterfrom
fix
Jan 19, 2024
Merged

Add filter callback to KeysetPaginator and new method Compare::withValue()#153
vjik merged 14 commits intomasterfrom
fix

Conversation

@vjik
Copy link
Member

@vjik vjik commented Jan 16, 2024

Q A
Is bugfix?
New feature? ✔️
Breaks BC?

@codecov
Copy link

codecov bot commented Jan 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (41f0d3d) 100.00% compared to head (f562b1d) 100.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##              master      #153   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity       254       261    +7     
===========================================
  Files             35        36    +1     
  Lines            634       668   +34     
===========================================
+ Hits             634       668   +34     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vjik vjik added the status:under development Someone is working on a pull request. label Jan 16, 2024
vjik and others added 3 commits January 16, 2024 16:34
@vjik vjik added status:code review The pull request needs review. and removed status:under development Someone is working on a pull request. labels Jan 16, 2024
@vjik vjik requested a review from a team January 16, 2024 13:37
@vjik vjik added status:under discussion status:under development Someone is working on a pull request. and removed status:code review The pull request needs review. labels Jan 16, 2024
@vjik vjik changed the title Add value caster to KeysetPaginator Add filter callback to KeysetPaginator and new method Compare::withValue() Jan 16, 2024
@vjik vjik added status:code review The pull request needs review. and removed status:under development Someone is working on a pull request. status:under discussion labels Jan 16, 2024
@vjik vjik merged commit 562c330 into master Jan 19, 2024
@vjik vjik deleted the fix branch January 19, 2024 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants