Skip to content

bug(nimbus): only highlight one timeline state at a time#13176

Merged
jaredlockhart merged 1 commit intomainfrom
13136
Aug 1, 2025
Merged

bug(nimbus): only highlight one timeline state at a time#13176
jaredlockhart merged 1 commit intomainfrom
13136

Conversation

@jaredlockhart
Copy link
Collaborator

Becuase

  • An experiment can only be in one state at a time
  • The timeline should clearly reflect this
  • We hit a bug where both enrolling and observing were highlighted simultaneously

This commit

  • Changes the definition of is_observation to only be true when an experiment is live and paused

fixes #13136

Becuase

* An experiment can only be in one state at a time
* The timeline should clearly reflect this
* We hit a bug where both enrolling and observing were highlighted simultaneously

This commit

* Changes the definition of is_observation to only be true when an experiment is live and paused

fixes #13136
@yashikakhurana
Copy link
Contributor

@jaredlockhart can we wait to merge this, I am testing it locally as it is related to launch flow

@jaredlockhart jaredlockhart disabled auto-merge July 31, 2025 21:23
Copy link
Contributor

@yashikakhurana yashikakhurana left a comment

Choose a reason for hiding this comment

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

Okay all good, thank you, tested lot of scenarios all worked, thank you @jaredlockhart

@jaredlockhart jaredlockhart added this pull request to the merge queue Aug 1, 2025
Merged via the queue into main with commit 7aceda1 Aug 1, 2025
15 checks passed
@jaredlockhart jaredlockhart deleted the 13136 branch August 1, 2025 16:11
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.

Both enrollment and observation are highlighted during review

3 participants