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

feat(protocol-designer, components): add retract for aspirate and dispense to PD #17411

Open
wants to merge 17 commits into
base: edge
Choose a base branch
from

Conversation

syao1226
Copy link
Collaborator

@syao1226 syao1226 commented Feb 4, 2025

re AUTH-906, AUTH-907

Overview

In this PR, I created a new MultiInputField component for the Submerge and Retract sections in the transfer steps, as they share a similar UI design with multiple InputStepFormFields. Additionally, I updated the PositionField and added a new type to the ListButton component to support a white button for the updated well position button design.

Test Plan and Hands on Testing

Changelog

  • Created a new MultiInputField component
  • Added a new type onColor to ListButton
  • Added new prefix values in PositionField and added retract xyz position fields to form-types
  • Added new retract fields to 8_5_0 migration file and updated the fixtures
  • Added unit tests

Review requests

Risk assessment

Copy link

codecov bot commented Feb 4, 2025

Codecov Report

Attention: Patch coverage is 71.93676% with 71 lines in your changes missing coverage. Please review.

Project coverage is 27.33%. Comparing base (731fb9e) to head (b152e8c).

Files with missing lines Patch % Lines
...s/MoveLiquidTools/MultipleStepsMoveLiquidTools.tsx 0.00% 41 Missing ⚠️
...ocolSteps/StepForm/PipetteFields/PositionField.tsx 0.00% 26 Missing ⚠️
protocol-designer/src/load-file/migration/8_5_0.ts 0.00% 4 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             edge   #17411      +/-   ##
==========================================
+ Coverage   26.23%   27.33%   +1.10%     
==========================================
  Files        3198     3199       +1     
  Lines      231041   231249     +208     
  Branches     9879    10433     +554     
==========================================
+ Hits        60617    63219    +2602     
+ Misses     170399   168001    -2398     
- Partials       25       29       +4     
Flag Coverage Δ
app 3.19% <0.00%> (-0.03%) ⬇️
components 4.34% <2.38%> (?)
labware-library 3.94% <1.58%> (-0.03%) ⬇️
protocol-designer 17.59% <71.93%> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
components/src/atoms/ListButton/index.tsx 95.55% <100.00%> (+0.68%) ⬆️
protocol-designer/src/form-types.ts 80.64% <ø> (ø)
...-designer/src/organisms/TipPositionModal/index.tsx 90.47% <100.00%> (+0.03%) ⬆️
...-designer/src/organisms/TipPositionModal/utils.tsx 90.41% <100.00%> (+0.27%) ⬆️
...Form/StepTools/MoveLiquidTools/MultiInputField.tsx 100.00% <100.00%> (ø)
protocol-designer/src/resources/types.ts 100.00% <100.00%> (ø)
...src/step-forms/test/createPresavedStepForm.test.ts 100.00% <100.00%> (ø)
...r/src/steplist/formLevel/getDefaultsForStepType.ts 93.26% <100.00%> (+0.36%) ⬆️
...list/formLevel/test/getDefaultsForStepType.test.ts 100.00% <100.00%> (ø)
...tocol-designer/src/ui/steps/test/selectors.test.ts 100.00% <100.00%> (ø)
... and 3 more

... and 90 files with indirect coverage changes

@syao1226 syao1226 changed the title feat(protocol-designer): add retract for aspirate and dispense to PD feat(protocol-designer, components): add retract for aspirate and dispense to PD Feb 7, 2025
@syao1226 syao1226 marked this pull request as ready for review February 10, 2025 20:44
@syao1226 syao1226 requested review from a team as code owners February 10, 2025 20:44
@syao1226 syao1226 requested review from smb2268, koji and ncdiehl11 and removed request for a team and smb2268 February 10, 2025 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants