Observation/FOUR-14185: Pagination does not work if we have a filter with more than one page. by agustinbusso · Pull Request #6249 · ProcessMaker/processmaker
Navigation Menu
{{ message }}
ProcessMaker / processmaker Public
- Notifications You must be signed in to change notification settings
- Fork 246
Merged
Conversation
Copy link
Contributor
Issue & Reproduction Steps
Steps to reproduced:
- Log in
- Go to request/all
- Select a process/status/start filter
- Perform a search that has more than one page
- Apply the filter
- Click on pagination-button
Solution
- Add a flag to not go to first page when using navigate
Working video
Screen.Recording.2024-02-19.at.15.58.09.mov
How to Test
- Follow steps above
- Also test it's working the scenario described here: https://processmaker.atlassian.net/browse/FOUR-14155
Related Tickets & Packages
Code Review Checklist
- I have pulled this code locally and tested it on my instance, along with any associated packages.
- This code adheres to ProcessMaker Coding Guidelines.
- This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
- This solution fixes the bug reported in the original ticket.
- This solution does not alter the expected output of a component in a way that would break existing Processes.
- This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
- This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
- This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
- This ticket conforms to the PRD associated with this part of ProcessMaker.
ci:next
Copy link
processmaker-sonarqube
bot
commented
Feb 19, 2024
processmaker-sonarqube bot commented
Feb 19, 2024
Quality Gate passed
Kudos, no new issues were introduced!
0 New issues
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code
fagubla approved these changes Feb 19, 2024
ryancooley
merged commit
8801eb7
into
next
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment