Skip to content

[bot] Merge 26.3 to 26.7#726

Merged
github-actions[bot] merged 2 commits into
release26.7-SNAPSHOTfrom
26.7_fb_bot_merge_26.3
Jul 20, 2026
Merged

[bot] Merge 26.3 to 26.7#726
github-actions[bot] merged 2 commits into
release26.7-SNAPSHOTfrom
26.7_fb_bot_merge_26.3

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Generated automatically.
Merging changes from: 06e8031
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.7_fb_bot_merge_26.3

labkey-martyp and others added 2 commits July 14, 2026 05:38
…equired state (#721)

## Rationale

A death or departure record could be finalized while other data for the
animal was still in the Review Required QC state, leaving unreviewed
records behind for an animal no longer at the center. Completing a death
or departure now fails validation until that data is reviewed, except
for records on the completing record's own task, which move to Completed
in the same save.

## Related Pull Requests

None.

## Changes

- Added the `study.studyDataReviewRequired` module query
(`study.StudyData` joined to `core.QCState`) listing records in Review
Required state.
- Added `NIRC_EHRTriggerHelper.getReviewRequiredDatasets`, returning the
dataset labels still in Review Required for an animal, excluding records
on the given taskid.
- The `deaths.js` and `departure.js` triggers now block saving a record
as Completed while review-required data exists, naming the offending
dataset(s) in the error.
- Added `NIRC_EHRTest.testDeathAndDepartureBlockedByReviewRequiredData`
covering the blocked path and the own-task exclusion for both datasets
via validate-only API saves.
@github-actions
github-actions Bot merged commit 717c5a8 into release26.7-SNAPSHOT Jul 20, 2026
7 of 8 checks passed
@github-actions
github-actions Bot deleted the 26.7_fb_bot_merge_26.3 branch July 20, 2026 14:29
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.

3 participants