-
Notifications
You must be signed in to change notification settings - Fork 180
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(app): update Select pipette path component #18047
base: edge
Are you sure you want to change the base?
Conversation
update Select pipette path component to align with the design close AUTH-
|
||
## [WIP] Version 1.2.0 | ||
|
||
Due to changes in the Quick Transfer setup flow, there will be changes to QuickTransferWizardState and QuickTransferSummaryState. The changes are as follows: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i haven't updated QuickTransferSummaryState because we need to keep the current setup flow and summry. the items in QuickTransferWizardState are optional so this change doesn't have any impact on the current setup flow.
// ToDo (kk:04/10/2025) uncomment this when add tip frequency | ||
// case 'SET_CHANGE_TIP': { | ||
// return { | ||
// ...state, | ||
// changeTip: action.changeTip, | ||
// } | ||
// } | ||
// ToDo (kk:04/10/2025) uncomment this when add drop tip location | ||
// case 'SET_DROP_TIP_LOCATION': { | ||
// return { | ||
// ...state, | ||
// dropTipLocation: action.location, | ||
// } | ||
// } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these will be uncomment when i add each component.
@@ -23,6 +23,8 @@ export interface QuickTransferWizardState { | |||
destinationWells?: string[] | |||
transferType?: TransferType | |||
volume?: number | |||
// Note added for liquid classes in Quick Transfer |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this comment will be removed when ff is removed
|
Overview
update Select pipette path component to align with the design
close AUTH-1003
Test Plan and Hands on Testing
Changelog
Review requests
Risk assessment
low