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

refactor(labware-creator): Break out well shape and depth sections #7794

Merged
merged 1 commit into from
May 11, 2021

Conversation

Kadee80
Copy link
Contributor

@Kadee80 Kadee80 commented May 11, 2021

Overview

addresses #7707 by breaking out Well Shape/Sides and Well Bottom/Depth

Changelog

  • refactor(labware-creator): Break out well shape and depth sections
  • add tests

Review requests

  • LC still works like production
  • Code review

Risk assessment

Low but lets make sure its matched production LC

@Kadee80 Kadee80 requested review from a team, shlokamin and IanLondon and removed request for a team May 11, 2021 18:42
Copy link
Contributor

@IanLondon IanLondon left a comment

Choose a reason for hiding this comment

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

LGTM.

Note to self mostly: we'll probably have to replace displayAsTube with something that handles more cases (tube vs non-tube not enough to accommodate plate/tube/tiprack conditionality)

@Kadee80 Kadee80 merged commit 0b631a3 into edge May 11, 2021
@Kadee80 Kadee80 deleted the lc_well-shape branch May 11, 2021 21:13
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