Display firmware version for logs if available#902
Merged
1Blademaster merged 1 commit intoDec 14, 2025
Conversation
Member
1Blademaster
commented
Dec 5, 2025
1Blademaster
commented
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds firmware version display to the Flight Log Analyser interface when the version information is available in the log file. The firmware version is extracted from the VER message's FWS field and conditionally displayed in the UI.
Key Changes:
- Adds firmware version extraction from VER message in log files
- Integrates firmware version into Redux state management
- Conditionally displays firmware version in the main display UI
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| gcs/electron/fla.ts | Extracts firmware version from VER message FWS field with proper null checking and adds it to the log summary |
| gcs/electron/types/flaTypes.ts | Adds firmwareVersion field (string | null) to LogSummary interface |
| gcs/src/redux/slices/logAnalyserSlice.js | Adds firmwareVersion state, reducer, selector, and exports to Redux slice |
| gcs/src/fla.jsx | Dispatches firmware version to Redux store during log summary processing and reorganizes imports alphabetically |
| gcs/src/components/fla/mainDisplay.jsx | Displays firmware version in UI when available, removes truncation styling from Aircraft Type field for consistency |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
1Blademaster
deleted the
901-display-firmware-version-from-dataflash-logs
branch
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.