Skip to content

Remove mergesync and ogasync modules - #1839

Open
labkey-martyp wants to merge 1 commit into
release26.3-SNAPSHOTfrom
26.3_fb_remove_mergesync_ogasync
Open

Remove mergesync and ogasync modules#1839
labkey-martyp wants to merge 1 commit into
release26.3-SNAPSHOTfrom
26.3_fb_remove_mergesync_ogasync

Conversation

@labkey-martyp

@labkey-martyp labkey-martyp commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

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.

Related Pull Requests

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.

Deletes both modules and the references they left behind in onprc_ehr.

LabworkRequestFormType declared a client dependency on MergeSync; ClientDependency.supplierFromModuleName throws IllegalArgumentException when the named module is absent, and this form type is registered, so it would have failed at runtime.

The mergeSyncInfo/automaticresults column came from the mergesync table customizer, so it is dropped from the clinpathRuns Requests custom view. The two EnterDataPanel links that filtered on that column collapse into a single Requests link now that manual and automatic results are indistinguishable.

LabworkRequestInstructionsFormSection and LabworkRequestInstructionsPanel.js rendered Merge instructions and nothing else; the form section was already commented out of LabworkRequestFormType.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant