Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Y24-194 [PR] Fix LCMT DNA Adp Lig purpose record #4608

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

yoldas
Copy link
Member

@yoldas yoldas commented Jan 16, 2025

Closes #

Changes proposed in this pull request

  • LCMT DNA Adp Lig is not an input plate; its state is not determined by a submission build on it. This update fixes the state transition from pending to started at Bravo Verify Initial Setup and started to passed in Manual Transfer.

Instructions for Reviewers

[All PRs] - Confirm PR template filled
[Feature Branches] - Review code
[Production Merges to main]
    - Check story numbers included
    - Check for debug code
    - Check version

@yoldas yoldas self-assigned this Jan 16, 2025
Copy link

codecov bot commented Jan 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.25%. Comparing base (6a728b5) to head (21ad54b).
Report is 150 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4608      +/-   ##
===========================================
- Coverage    89.25%   89.25%   -0.01%     
===========================================
  Files         1406     1406              
  Lines        30053    30053              
===========================================
- Hits         26824    26823       -1     
- Misses        3229     3230       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yoldas yoldas requested a review from andrewsparkes January 16, 2025 10:53
@yoldas yoldas merged commit 132bca6 into develop Jan 16, 2025
25 checks passed
@yoldas yoldas deleted the y24-194-fix-lcmt-dna-adp-lig-purpose-record branch January 16, 2025 11:14
@yoldas yoldas added the EMSeq emSEQ label Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EMSeq emSEQ
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants