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.

Sign up for GitHub

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

Jump to bottom

Wire up Heater Shaker Wizard TestShake page and slideout #9613

Closed
sakibh opened this issue Mar 2, 2022 · 0 comments · Fixed by #9833
Closed

Wire up Heater Shaker Wizard TestShake page and slideout #9613

sakibh opened this issue Mar 2, 2022 · 0 comments · Fixed by #9833
Assignees
@sakibh
Labels
app Affects the `app` project heater-shaker heater shaker software support

Comments

@sakibh
Copy link
Contributor

sakibh commented Mar 2, 2022 •
edited
Loading

Overview

The TestShake component of the Heater Shaker Wizard requires rendering dynamically and is currently stubbed out. It will likely need to accept the heater shaker module as a prop to pass the necessary data to the child components.

Acceptance Criteria

  • Dynamically render banner text: if there is a protocol the plate name should come from the labware definition.
  • Disable Start Shaking button if the labware latch is open
  • Wire up Start Shaking button
  • Wire up Open Labware Latch button
  • Wire up set shake speed input
  • Wire up the min/max values from either the labware definition or default values from the module
  • Wire up HeaterShakerModuleCard component being used in the TestShake component with data from the API.
  • Add labware latch statuses to i18n.
@anthonywilleee anthonywilleee added app Affects the `app` project heater-shaker heater shaker software support labels Mar 8, 2022
@sakibh sakibh changed the title Wire up Heater Shaker Wizard TestShake page Wire up Heater Shaker Wizard TestShake page and slideout Mar 10, 2022
@sakibh sakibh self-assigned this Mar 10, 2022
sakibh added a commit that referenced this issue Mar 30, 2022
@sakibh
refactor(app): wire up heater shaker wizard TestShake page
c5e9055 
This PR wires up the buttons and input for the testshake page of the heater shaker wizard.

closes #9613
@sakibh sakibh mentioned this issue Mar 30, 2022
Merged
sakibh added a commit that referenced this issue Apr 7, 2022
@sakibh
refactor(app): wire up heater shaker wizard TestShake page (#9833)
1f34210 
* refactor(app): wire up heater shaker wizard TestShake page

This PR wires up the buttons and input for the testshake page of the heater shaker wizard.

closes #9613
ecormany pushed a commit that referenced this issue Apr 8, 2022
@sakibh @ecormany
refactor(app): wire up heater shaker wizard TestShake page (#9833)
7a81d43 
* refactor(app): wire up heater shaker wizard TestShake page

This PR wires up the buttons and input for the testshake page of the heater shaker wizard.

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

@sakibh sakibh

Labels
app Affects the `app` project heater-shaker heater shaker software support
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor(app): wire up heater shaker wizard TestShake page Opentrons/opentrons
2 participants
@sakibh @anthonywilleee