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

fix(app): fix inescapable pipette wizard #17008

Merged
merged 1 commit into from
Dec 3, 2024

Conversation

mjhuff
Copy link
Contributor

@mjhuff mjhuff commented Dec 2, 2024

Closes RABR-675

Overview

It's possible to sometimes get stuck in the pipette wizard without an exit button if the error occurs while a command is executing or the maintenance run is trying to be deleted. Additionally (and I think this is how the bug ABR reported happens to begin with) - if an error does occur once during pipette flows, we were just closing the wizard without deleting the maintenance run. handleCleanUpAndClose conditionally closes the run if it does exist, which we definitely want to do if we error.

Lastly, it's probably better to show an exit button while certain requests are pending and just disable it rather than avoid rendering an exit button altogether.

This control flow more closely follows what happens in GripperWizardFlows, which is probably why ABR & QA haven't seen this bug crop up in that flow.

Test Plan and Hands on Testing

  • Ran a bunch of pipette wizard flows, verifying intended behavior.

Changelog

  • Fixed a bug in which it's possible not to escape pipette wizard flows.

Risk assessment

low

@mjhuff mjhuff requested review from sfoster1, TamarZanzouri, smb2268 and a team December 2, 2024 20:04
@mjhuff mjhuff requested a review from a team as a code owner December 2, 2024 20:04
@mjhuff mjhuff force-pushed the app_fix-inescapable-pipette-wizard branch 4 times, most recently from 9413276 to 189f6ca Compare December 2, 2024 20:12
Copy link
Member

@sfoster1 sfoster1 left a comment

Choose a reason for hiding this comment

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

None escape the wizard

@mjhuff mjhuff merged commit 91d737d into edge Dec 3, 2024
30 checks passed
@mjhuff mjhuff deleted the app_fix-inescapable-pipette-wizard branch December 3, 2024 15:20
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