Skip to content

Y26-021 Accessioning - removed accessioning check from work completion#5538

Open
andrewsparkes wants to merge 4 commits intodevelopfrom
y26-021-accessioning-remove-check-at-charge-and-pass
Open

Y26-021 Accessioning - removed accessioning check from work completion#5538
andrewsparkes wants to merge 4 commits intodevelopfrom
y26-021-accessioning-remove-check-at-charge-and-pass

Conversation

@andrewsparkes
Copy link
Member

@andrewsparkes andrewsparkes commented Feb 11, 2026

Closes #5481

See also related deployment PR for removal of flag:
https://github.com/sanger/deployment/pull/800

Changes proposed in this pull request

  • Removed accessioning check at work completion and when submitting for sequencing using a SequencingRequest.

Instructions for Reviewers

Deletion of unneeded code.

@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.30%. Comparing base (2f9b4d6) to head (6e0511d).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5538      +/-   ##
===========================================
- Coverage    87.35%   87.30%   -0.06%     
===========================================
  Files         1463     1461       -2     
  Lines        33005    32974      -31     
  Branches      3467     3460       -7     
===========================================
- Hits         28833    28787      -46     
- Misses        4151     4166      +15     
  Partials        21       21              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@StephenHulme StephenHulme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants