Skip to content

798 gcs connection announcement#799

Merged
1Blademaster merged 8 commits intomainfrom
798-gcs-connection-announcement
Oct 29, 2025
Merged

798 gcs connection announcement#799
1Blademaster merged 8 commits intomainfrom
798-gcs-connection-announcement

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings October 29, 2025 18:26
@1Blademaster 1Blademaster linked an issue Oct 29, 2025 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds flight software version detection and validation during drone connection, along with UI improvements for status messages. The system now checks that the connected autopilot is running ArduPilot 4.x.x and displays the version to the user.

  • Implements flight software version detection via MAVLink AUTOPILOT_VERSION message
  • Adds version validation to reject unsupported versions (only 4.x.x supported)
  • Improves status message UI to work when not connected to drone
  • Updates MAVLink connection to properly identify as a GCS

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
radio/app/utils.py Adds utility functions to decode and format flight software version from MAVLink message
radio/app/drone.py Implements version detection during initialization with proper validation and error handling
radio/app/endpoints/comPorts.py Includes flight software version in connection response to frontend
radio/app/endpoints/states.py Removes STATUSTEXT from endpoint message list (now handled as always-on listener)
gcs/src/redux/slices/droneInfoSlice.js Adds Redux state for storing flight software version
gcs/src/redux/slices/statusTextSlice.js Adds action to reset messages on new connection
gcs/src/redux/middleware/socketMiddleware.js Dispatches flight software version and resets messages on connection
gcs/src/dashboard.jsx Updates status messages panel to display when not connected
gcs/src/components/dashboard/statusMessages.jsx Handles messages without timestamps for non-connected state

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread radio/app/utils.py Outdated
Comment thread radio/app/drone.py Outdated
Comment thread radio/app/drone.py Outdated
Comment thread radio/app/drone.py Outdated
@1Blademaster 1Blademaster requested a review from a team October 29, 2025 18:33
NexInfinite
NexInfinite previously approved these changes Oct 29, 2025
@1Blademaster 1Blademaster merged commit cdafe72 into main Oct 29, 2025
8 checks passed
@1Blademaster 1Blademaster deleted the 798-gcs-connection-announcement branch October 29, 2025 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GCS connection announcement

3 participants