Timer v2: Emit timer value from machine #161
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This is currently rendering "15" on the page (for the initial timer number). We need to decrease it by 1 each second. This is currently separate from the XState countdown timer, but there might be a way to read this value in future. This is a first step. We've spotted a bug where this is currently rendering multiple times, but we'll solve that soon.
We had to remove this to get the "countdown" switch statement to work, but maybe there's a way to get it working.
We feel keeping a separate implementation of timer in client and server could be a bit buggy. This is an investigation into tracking a timer value using state machines: - Added a timer machine - invoked machine from round machine on question answered - log the value in the console when the timer is running. The timer doesn't currently stop.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a second attempt at getting the timer working - but emitting it from the state machine rather than managing it from client