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 never clicking setup tab after the run has started #16072

Merged
merged 1 commit into from
Aug 20, 2024

Conversation

mjhuff
Copy link
Contributor

@mjhuff mjhuff commented Aug 20, 2024

Closes RQA-3065 and RQA-3059

Overview

We recently updated the routing logic to the setup tab, but the useEffect dependency array is a bit overzealous. Let's only do the redirect on the first click only.

Current Behavior

Screen.Recording.2024-08-20.at.2.57.42.PM.mov

Fixed Behavior

Screen.Recording.2024-08-20.at.2.58.19.PM.mov

Test Plan and Hands on Testing

  • See videos.

Changelog

  • Fixed never being able to click on Setup after the run has started.

Risk assessment

lowwww

@mjhuff mjhuff requested a review from ncdiehl11 August 20, 2024 19:00
@mjhuff mjhuff requested a review from a team as a code owner August 20, 2024 19:00
Copy link
Collaborator

@ncdiehl11 ncdiehl11 left a comment

Choose a reason for hiding this comment

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

looks right to me

@mjhuff mjhuff force-pushed the app_fix-no-run-setup branch from 9001de1 to 76fbbec Compare August 20, 2024 19:16
@mjhuff mjhuff merged commit ba8692a into chore_release-8.0.0 Aug 20, 2024
20 checks passed
@mjhuff mjhuff deleted the app_fix-no-run-setup branch August 20, 2024 19:50
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