-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Visit history - correct count for displaying ALL results on a page #7386
Conversation
cc08220
to
4c76626
Compare
@ruthkonyn unfortunately this PR has conflicts with encounter. I hoped to get in patch. Maybe you can get resolved early Monday morning before I build the patch! |
Doesn't make sense for Styling to fail after I made and merged a fixed file so either git is trying to merge an older file(a no no) or actions is testing the older version. |
Okay looks like actions is based on the last master merge into branch otherwise it should have picked up my merged changes. |
thank you @sjpadgett - so there is nothing i need to do now :-) . Thanks for accepting into the patch |
…penemr#7386) * display correct count values for ALL - issue openemr#7385 * delete blank line * escape strings on line 317 * correct style --------- Co-authored-by: Jerry Padgett <sjpadgett@gmail.com> (cherry picked from commit 885ee27)
Fixes #7385
when displaying all the results on a page, having previously displayed a shorter page, give the correct count of results being displayed
Changes proposed in this pull request: