Skip to content

/ javers

New issue

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

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

Already on GitHub? Sign in to your account

Jump to bottom

Pagination for .findSnapshots / .findChanges #179

Closed
fynnfeldpausch opened this issue Jul 23, 2015 · 4 comments
Closed

Pagination for .findSnapshots / .findChanges #179

fynnfeldpausch opened this issue Jul 23, 2015 · 4 comments
Assignees
@mwesolowski

Comments

@fynnfeldpausch
Copy link

@fynnfeldpausch fynnfeldpausch commented Jul 23, 2015

Is there any way to use a pagination filter when searching snapshots? The QueryBuilder provides a .limit() functionality but I am not able to find any .skip() feature. Both methods are needed to implement a proper pagination for searching snapshots.
@bartoszwalacik
Copy link
Member

@bartoszwalacik bartoszwalacik commented Jul 23, 2015

hi, there is no pagination yet, agreed that it could be useful. If you want to have it quickly, think about contribution
@bartoszwalacik bartoszwalacik mentioned this issue Sep 30, 2015
Closed
@mwesolowski
Copy link
Member

@mwesolowski mwesolowski commented Dec 26, 2015

Hi, I will add skip functionality to QueryBuilder when PR #282 is accepted and merged. Cheers.
@bartoszwalacik bartoszwalacik added review and removed in progress labels Jan 10, 2016
@bartoszwalacik
Copy link
Member

@bartoszwalacik bartoszwalacik commented Jan 17, 2016

PR in review #285
bartoszwalacik added a commit that referenced this issue Jan 17, 2016
@bartoszwalacik
#179 small style fixes
Loading status checks…
0ee047e
bartoszwalacik added a commit that referenced this issue Jan 17, 2016
@bartoszwalacik
Merge pull request #308 from javers/mwesolowski-179-skipping-query-re…
Loading status checks…
b7c9154 
…qults

#179 small style fixes
@bartoszwalacik bartoszwalacik added review and removed in progress labels Jan 17, 2016
@bartoszwalacik bartoszwalacik removed the review label Jan 18, 2016
@bartoszwalacik
Copy link
Member

@bartoszwalacik bartoszwalacik commented Jan 18, 2016

released in 1.4.3
see http://javers.org/documentation/jql-examples/#skip-filter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mwesolowski mwesolowski

Labels
None yet
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@fynnfeldpausch @bartoszwalacik @mwesolowski