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: opentrons ai client Instruments Section #16646

Merged
merged 13 commits into from
Nov 4, 2024

Conversation

fbelginetw
Copy link
Collaborator

@fbelginetw fbelginetw commented Oct 30, 2024

Overview

This PR adds the Instruments section to the create protocol flow.
It adds the Exit button and its functionality to the flows.

image

Test Plan and Hands on Testing

  • On the landing page click Create a new protocol button, you will be redirected to the new page
  • fill up the required information in the Application section and click Confirm
  • You now can select different Robots and pipettes
  • The Prompt Preview component is updated with the data entered.

Changelog

  • Add second section Instruments
  • Now on the create and update protocol flows there's an exit button instead of Logout
  • Exit button displays a confirm Modal

Review requests

  • Verify new section.
  • Verify Exit functionality

Risk assessment

  • low

@fbelginetw fbelginetw marked this pull request as ready for review October 30, 2024 20:18
@fbelginetw fbelginetw requested review from a team as code owners October 30, 2024 20:18
@fbelginetw fbelginetw requested review from TamarZanzouri, koji, y3rsh, shlokamin and Elyorcv and removed request for a team October 30, 2024 20:18
@fbelginetw fbelginetw merged commit 185aeb8 into edge Nov 4, 2024
34 checks passed
@fbelginetw fbelginetw deleted the opentrons-ai-client-instruments branch November 4, 2024 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants