-
Notifications
You must be signed in to change notification settings - Fork 179
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
Conversation
…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
Codecov Report
Additional details and impacted files@@ 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
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
nice this makes sense to me!
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.
Let's rename the const but then this lgtm!
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.
4️⃣
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
Changelog
Review requests
@brenthagen
Risk assessment
low