Display seconds in CommsSection timestamps (fixes #491)#509
Display seconds in CommsSection timestamps (fixes #491)#509ksalamy merged 2 commits intombari-org:developfrom
Conversation
Extends the full-resolution timestamp display from LogsSection to CommsSection. Command and mission event timestamps now show H:mm:ss for correlation with other datastreams and sources.
There was a problem hiding this comment.
Pull request overview
Extends full-resolution timestamp formatting (including seconds) to the CommsSection so comms queue events can be correlated with other time-based data sources, matching the existing behavior in LogsSection.
Changes:
- Update CommsSection time formatting from
H:mmtoH:mm:ssfor comms event timestamps.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| const day = today | ||
| ? 'Today' | ||
| : DateTime.fromISO(item?.commsIsoTime ?? '').toFormat('MMM d yyyy') | ||
| const time = DateTime.fromISO(item?.commsIsoTime ?? '').toFormat('H:mm') | ||
| const time = DateTime.fromISO(item?.commsIsoTime ?? '').toFormat('H:mm:ss') | ||
|
|
There was a problem hiding this comment.
CommsSection has existing component tests, but none assert the rendered timestamp format. Since this change is specifically about showing seconds, please add/extend a test to verify the time text includes seconds (e.g., matches /\d{1,2}:\d{2}:\d{2}/) for a known mocked commsIsoTime value, so regressions don’t slip in unnoticed.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Verifies timestamps include seconds for correlation with other datastreams.
Extends the full-resolution timestamp display from LogsSection to CommsSection. Command and mission event timestamps now show H:mm:ss for correlation with other datastreams and sources.
Addresses #491.