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(components): render 4th column slot label for staging area slots #13969

Merged
merged 7 commits into from
Nov 16, 2023

Conversation

ncdiehl11
Copy link
Collaborator

@ncdiehl11 ncdiehl11 commented Nov 13, 2023

closes RAUT-782

Overview

Creates a 4th column in SlotLabels component to be rendered if used for coordinates for slot positions AND at least one staging area fixture exists in deck config. Note that SlotLabels do not render 4th column when rendered as coordinates for fixture positions.

Test Plan

  • Verify that 4th column renders in BaseDeck instances where DeckConfiguration includes >1 staging area slot
  • NOTE: further testing blocked by inability to test a protocol including a staging area slot
Screen Shot 2023-11-16 at 1 13 23 PM Screen Shot 2023-11-16 at 1 15 10 PM

Changelog

  • add 4th column label to SlotLabels (renders conditionally on cases specified above)
  • determine presence of staging areas in BaseDeck component from deckConfig, and render 4th column slot label based on this condition being true
  • ensure slot labels only render columns 1-3 for DeckLocationSelect

Review requests

@brenthagen

Risk assessment

low

…nt in deck config

Creates a 4th column in SlotLabels component to be rendered if used for coordinates for slot
positions AND at least one staging area fixture exists in deck config. Note that SlotLabels do not
render 4th column when rendered as coordinates for fixture positions
@ncdiehl11 ncdiehl11 requested a review from a team as a code owner November 13, 2023 20:04
@ncdiehl11 ncdiehl11 requested review from jerader and removed request for a team November 13, 2023 20:04
Copy link

codecov bot commented Nov 13, 2023

Codecov Report

Merging #13969 (d4cb82a) into edge (e4c123f) will decrease coverage by 0.07%.
Report is 10 commits behind head on edge.
The diff coverage is 37.50%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             edge   #13969      +/-   ##
==========================================
- Coverage   70.67%   70.61%   -0.07%     
==========================================
  Files        2505     2506       +1     
  Lines       70883    70923      +40     
  Branches     8756     8798      +42     
==========================================
- Hits        50097    50080      -17     
- Misses      18644    18694      +50     
- Partials     2142     2149       +7     
Flag Coverage Δ
app 67.57% <ø> (-0.20%) ⬇️
components 62.56% <37.50%> (-0.11%) ⬇️
labware-library 51.50% <ø> (ø)
protocol-designer 45.67% <ø> (ø)

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

Files Coverage Δ
...onents/src/hardware-sim/DeckConfigurator/index.tsx 73.07% <ø> (ø)
components/src/hardware-sim/BaseDeck/BaseDeck.tsx 0.00% <0.00%> (ø)
components/src/hardware-sim/Deck/SlotLabels.tsx 44.44% <42.85%> (-5.56%) ⬇️

... and 24 files with indirect coverage changes

Copy link
Collaborator

@jerader jerader left a comment

Choose a reason for hiding this comment

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

nice this makes sense to me!

@ncdiehl11 ncdiehl11 requested a review from jerader November 13, 2023 21:25
Copy link
Collaborator

@jerader jerader left a comment

Choose a reason for hiding this comment

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

Let's rename the const but then this lgtm!

@ncdiehl11 ncdiehl11 requested a review from brenthagen November 16, 2023 18:16
Copy link
Contributor

@brenthagen brenthagen left a comment

Choose a reason for hiding this comment

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

4️⃣

@ncdiehl11 ncdiehl11 merged commit 76cb53b into edge Nov 16, 2023
@ncdiehl11 ncdiehl11 deleted the feat_app-staging-area-slot-labels branch November 16, 2023 20:56
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.

3 participants