feat: Pull Request pagination #463
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| PullRequestPageRequest pageRequest) { | ||
| log.debug("Starting pagination process"); | ||
| log.debug("Input parameters - Filter: {}, Page: {}, Size: {}, Search Term: {}", | ||
| pageRequest.getFilterType(), pageRequest.getPage(), pageRequest.getSize(), pageRequest.getSearchTerm()); |
There was a problem hiding this comment.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> reported by reviewdog 🐶
Line is longer than 100 characters (found 112).
| log.debug("Pagination Details - Page Size: {}, Total Pages: {}", pageSize, totalPages); | ||
|
|
||
| // Prepare the final list of PRs for the current page | ||
| List<PullRequestBaseInfoDto> pageContent = new ArrayList<>(); |
There was a problem hiding this comment.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> reported by reviewdog 🐶
Distance between variable 'pageContent' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
| pageRequest.setFilterType(filterType != null ? filterType : PullRequestFilterType.ALL); | ||
| pageRequest.setSearchTerm(searchTerm); | ||
|
|
||
| PageResponse<PullRequestBaseInfoDto> response = pullRequestService.getPaginatedPullRequests(pageRequest); |
There was a problem hiding this comment.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> reported by reviewdog 🐶
Line is longer than 100 characters (found 109).
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferencesCodacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more Footnotes
|
|
There hasn't been any activity on this pull request recently. Therefore, this pull request has been automatically marked as stale and will be closed if no further activity occurs within seven days. Thank you for your contributions. |
Work in progress
Motivation
Description
Testing Instructions
Prerequisites:
Flow:
Screenshots
Checklist
General
Server
Client