Skip to content

[bot] Fast-forward for 26.7.1#729

Merged
github-actions[bot] merged 2 commits into
release26.7from
26.7_ff_bot_26.7.1
Jul 21, 2026
Merged

[bot] Fast-forward for 26.7.1#729
github-actions[bot] merged 2 commits into
release26.7from
26.7_ff_bot_26.7.1

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Generated automatically.
Approve all matching PRs simultaneously.
Approval will trigger automatic merge.
View all PRs: https://internal.labkey.com/Scrumtime/Backlog/harvest-gitOpenPullRequests.view?branch=26.7_ff_bot_26.7.1

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 Jul 21, 2026
17 checks passed
@github-actions
github-actions Bot deleted the 26.7_ff_bot_26.7.1 branch July 21, 2026 17:24
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