Open
Conversation
…o speed up queries more
…tch case instead of if else for performance, remove space seperator from date time string
…void potential NullPointerExceptions, add NumberFormatException handling inside of EctConsultationFormRequestUtil and ctViewConsultationRequestsUtil, prevent stale professionalSpecialist values by clearing contact fields if request is null, restore legacy date time matching, sanitize field before audit logging, added javadoc to FaxJobDao and FaxJobDaoImpl new methods
…rvices entities and their EAGER specialist collections
…alize blank providerNo's to ensure rows load correctly with privacy filters, return null provider to avoid outputting null provider name, populate all parallel lists in ectConsultationVecByDemographic to preven IndexOutOfBounds Exception, replace hardcoded extension keys for enum refs, add extra context to error log messages, return default sorting by ascending
…pages after it is all loaded no matter the limit per page, incorporated the changes to work with the previous DTO projection changes in the tickler manager page
…te is used in the queries and stays between searches
…a clamped start value to avoid negative or very large values, add null checks for dates, ensured that emtpy dates get set to todays date, or a large date when viewing through demographics to match previous behaviour, added try catches for possible exceptions, renamed warningCOlumn to rowTypeColumn for easier time understanding the use of it
…, allowing for consistent counts on all limit types
…r truncation when over max amount, add audit logging to tickler list, change exception throws to endpoint JSON responses for errors, refactored action to be more readable, ensured that comment row is reset before iterating over columns, adding unit testing to action class
…viewed, and account for legacy stored data by setting view dropdown if using the old options
… into specialists list calls
Move specialist data loading out of inline JSP scriptlets into a dedicated SpecialistList2Action that serves JSON via fetch() POST. The JSP pages now load a lightweight shell with spinner, then fetch and batch-render rows client-side using requestAnimationFrame.
…orJackson arrayNode creation
… for integration with devcontainer rebuilds
…ss REST TO models
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
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This branch packages 5 fixes and performance improvements.
Performance Fixes:
1. PR #2333 - Performance: slow edit specialists page (7 commits)
Related Issue: #2321
2. PR #2264 - Fix: slow consultation pages - LEGACY BUG (14 commits)
Related Issue: openo-beta#2099
3. PR #2268 - Performance/Improvement: tickler pagination not working correctly - LEGACY BUG (11 commits)
Related Issue: openo-beta#2259
Bug Fixes
4. PR #2332 - Fix: Update instructions/label for Asthma Action Plan measurement (6 commits)
Related Issue: #2319
5. PR #2334 - fix: restore epoch serialization for date fields across REST TO models to fix Ocean sync (1 commit)