ECHO-930 feat(participant): prevent double start recording and track actual recording start time - #990
Merged
Conversation
…cording start time Double-start guards in the portal: - ref latch on the initiate form (submit + skipOnboarding auto-submit) so a fast double click creates one conversation, not two; the auto-submit path now reuses startConversation and gains the missing conversation_started event - re-entrancy latch and cancellation token in useChunkedAudioRecorder: a second tap during the mic prompt is a no-op, stopping during the prompt releases the acquired stream, and a failed start rolls back recorder, stream, and meter state so retries keep working - recording_started analytics and the wake lock only fire when recording actually starts recording_started_at: - new nullable conversation.recording_started_at column (Directus snapshot) - primary source: the first liveness ping with state "recording" persists the server-clocked start (same signal the live monitor already uses); the first audio chunk is the fallback, clamped to [created_at, now + 5m] and stored UTC - one conditional PATCH enforces the rules in the filter itself: uploads are never stamped, the server-clocked liveness value may move the stamp earlier, and the client-clocked chunk value only fills an empty one - text chunks and dashboard uploads never stamp; the conversations BFF retries without the field if the column is not deployed yet - dashboard lists and the detail panel display recording_started_at with a created_at fallback; sorting stays on created_at (accepted trade-off) - translations added for nl, de, fr, es; cs, it, uk fall back to English
Contributor
|
Hi ! Thank you for contributing to Dembrane ECHO! Before we consider your Pull Request, we ask that you sign our Contributor License Agreement (CLA). This is only required for your first Pull Request. Please review the CLA, and sign it by adding your GitHub username to the contributors.yml file. Thanks! |
This file contains hidden or 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
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.
Double-start guards in the portal:
recording_started_at: