Skip to content

fix: handle live stats correctly during test restart - #8348

Open
pankajbindra91 wants to merge 1 commit into
monkeytypegame:masterfrom
pankajbindra91:fix/live-progress-custom-text
Open

fix: handle live stats correctly during test restart#8348
pankajbindra91 wants to merge 1 commit into
monkeytypegame:masterfrom
pankajbindra91:fix/live-progress-custom-text

Conversation

@pankajbindra91

Copy link
Copy Markdown

##Description

Fixes live stats handling during test restarts by ensuring the timer logic respects the restart state before updating the displayed value.

##Changes

  • Added restart-state handling to the live timer logic.
  • Prevented stale state from affecting live stats after a restart.
  • Preserved existing time-limited test behavior.
  • Verified the fix with custom text tests.

Testing

  • Ran the type-aware Ox lint check.
  • Result: **0 warnings, 0 errors. **
  • Tested the change locally with test restarts.

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Aug 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and also include the author name at the end inside parenthesis. It looks like your proposed title needs to be adjusted.

Details:

Title "fix: handle live stats correctly during test restart"
didn't match the configured pattern. Please ensure that the title
contains your name so that you can be credited in our changelog.

A correct version would look something like:

feat: add new feature (@github_username)
impr(quotes): add english quotes (@username)
fix(leaderboard): show user rank correctly (@user1, @user2, @user3)

@fehmer

fehmer commented Aug 17, 2026

Copy link
Copy Markdown
Member

Duplicate #8341

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants