[bot] Merge 26.9 to develop - #1887
Merged
Merged
Conversation
Changed the flags for assignment active days from 30 to 180
…#1866) ## Rationale `ONPRC_EHRTest.testSubmitButtonsDisabledDuringValidation` fails intermittently on the 26.3 ONPRC EHR suites and is blocking ONPRC work, so it is being disabled until the assertion can be reworked. The assertion is racy by construction: `Force Submit` carries `disableOn: 'SEVERE'`, so it is disabled only while validation is in flight and is re-enabled the instant validation completes. Clicking `More Actions` and waiting for the Ext menu to render costs a few seconds, which a fast validation pass beats, and the 10-second wait for the disabled menu item then can never succeed. Every failure is the same one — `ONPRC_EHRTest.java:2036` on `Force Submit` — while the sibling checks pass regardless because `Submit And Reload` and `Submit Final` stay disabled after validation whenever the form has WARN/ERROR rows. ## Changes - `@Ignore` on `testSubmitButtonsDisabledDuringValidation`, with the race recorded in the annotation message and a comment noting that it is blocking ONPRC work and should be re-enabled once the flakiness is resolved.
## Rationale Retires the mergesync and ogasync modules, which are no longer used. mergesync provided the Merge LIS integration for Clinpath requests; ogasync ran an OGA grant ETL and had no dependents. ## Changes - Deletes both modules. - Removes the Merge integration from the Clinpath labwork request form, along with its instructions panel. - Drops the Merge-derived column from the Clinpath requests view and collapses the request links that filtered on it into a single link. - Leaves the similarly named billing OGA table and ETL in place; despite the name they are unrelated to the removed module.
@brentlogan: these changes are an extension to the MHC typing storage and ETLs. They mostly impact the PRIMe-seq side, but will ultimately need to be deployed to PRIMe. Can you confirm what LK version PRIMe is running? I would like to deploy the PRIMe-seq parts of this first, stage data, and then add the PRIMe pieces. Because I think you update less frequently, this might just happen naturally.
Modified Medication Printable online reports to now include qcstate reporting column
Modified Blood Draw input form to allow users an option to enter "Additional Services" selections
Stored procedure clean up of single use programs
## Rationale FinanceNotification.writeResultTable builds the ONPRC charge summary report by concatenating editor-entered database values (financial analyst, project, alias/account, OGA project number, category) and their derived URLs directly into HTML with no escaping. That HTML is rendered verbatim into the LDK RunNotificationAction admin preview via HtmlString.unsafe and is also sent as the HTML email body, so a stored payload in any of those project/alias fields executed in the browser of any user who previewed the notification or received the email — a stored XSS with privilege-escalation potential toward admins. This is the ONPRC counterpart to the BillingNotification fix. DCMFinanceNotification was a registered subclass that overrode writeResultTable with a near-duplicate copy of the same unescaped report; rather than carry and fix a second copy, it is removed entirely. ## Changes - FinanceNotification: wrap every editor-entered value (financial analyst, project, account, project number, category) and its derived href URL in PageFlowUtil.filter in the per-financial-analyst tables, and filter the top category summary table (url and category), which was unescaped in this copy; add the org.labkey.api.util.PageFlowUtil import. - DCMFinanceNotification: remove the class entirely and drop its NotificationService registration and import from ONPRC_BillingModule.
Added new column called "Source" to the Birth Records report on the animal history page
## Rationale Consolidate and migrate SQL upgrade scripts in the last three ONPRC modules: - extscheduler - onprc_ehr_compliancedb - onprc_ssu All three move from `SupportedDatabases: mssql` / `ManageVersion: false` to `mssql, pgsql` with managed versions.
## Rationale Add PG scripts to match recently added SQL Server scripts. These are only needed for local dev/test databases that ran earlier versions of the PG bootstrap scripts. Clean up sla-0.000-25.000.sql - remove significant CREATE/DROP/CREATE blocks, inline ALTER statements
## Rationale Added original user defined Queries to Code Base ## Related Pull Requests - <!-- list of links to related pull requests (replace this comment) --> ## Changes 2 Queries and associated XML files NHP Cull NHPUUL <!-- list of standard tasks (remove this comment to enable) ## Tasks 📍 - [ ] Claude Code Review - [ ] Manual Testing - [ ] Test Automation - [ ] Verify Fix --> --------- Co-authored-by: Binal Patel <binalp@labkey.com>
## Rationale New linked schema to enable Art Core to see invoiced items in folder `/ONPRC/Admin/Finance` from their core folder. It is created as a template so other cores can override the filter to see their invoiced items from their core folders. EHR Ticket: EHR Issue 14760: Unit data of items invoiced to other Units ## Related Pull Requests - <!-- list of links to related pull requests (replace this comment) --> ## Changes - <!-- list of descriptions of changes that are worth noting (replace this comment) --> <!-- list of standard tasks (remove this comment to enable) ## Tasks 📍 - [ ] Claude Code Review - [ ] Manual Testing - [ ] Test Automation - [ ] Verify Fix --> --------- Co-authored-by: loganb <loganb@ohsu.edu>
_Generated automatically._ Merging changes from: a82c5d5 **Approve all matching PRs simultaneously.** **Approval will trigger automatic merge.** Verify all PRs before approving: https://internal.labkey.com/Scrumtime/Backlog/harvest-gitOpenPullRequests.view?branch=26.9_fb_bot_merge_26.7 --------- Co-authored-by: Lakshmi Kolli <69940873+kollil@users.noreply.github.com> Co-authored-by: Marty Pradere <martyp@labkey.com> Co-authored-by: github-actions <teamcity@labkey.com> Co-authored-by: bbimber <bbimber@gmail.com> Co-authored-by: Ohsudev <76500320+Ohsudev@users.noreply.github.com> Co-authored-by: Binal Patel <binalp@labkey.com> Co-authored-by: Lum <klum@labkey.com>
cnathe
approved these changes
Sep 9, 2026
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
11 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.
Generated automatically.
Merging changes from: e0a3c74
Approve all matching PRs simultaneously.
Approval will trigger automatic merge.
Verify all PRs before approving: https://internal.labkey.com/Scrumtime/Backlog/harvest-gitOpenPullRequests.view?branch=fb_bot_merge_26.9