Skip to content

Add mavlink forwarding on successful reboot#861

Merged
Turnlings merged 2 commits intomainfrom
860-add-mavlink-forwarding-when-rebooting-drone
Nov 20, 2025
Merged

Add mavlink forwarding on successful reboot#861
Turnlings merged 2 commits intomainfrom
860-add-mavlink-forwarding-when-rebooting-drone

Conversation

@Turnlings
Copy link
Copy Markdown
Contributor

@Turnlings Turnlings commented Nov 18, 2025

Passes the forwarding address to the new drone instance on reboot. Given this address, the init function then already handles the actual begin forwarding.

Copilot AI review requested due to automatic review settings November 18, 2025 22:58
@Turnlings Turnlings linked an issue Nov 18, 2025 that may be closed by this pull request
@Turnlings Turnlings self-assigned this Nov 18, 2025
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 automatic restoration of MAVLink forwarding after a successful autopilot reboot. When the drone reboots and was previously forwarding MAVLink messages, it will automatically resume forwarding to the configured address.

Key Changes:

  • Imports emitStartForwarding action and selectIsForwarding selector from droneConnectionSlice
  • Checks forwarding state after successful reboot and re-initiates forwarding if it was previously active

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

@Turnlings Turnlings requested a review from a team November 18, 2025 23:03
Comment thread gcs/src/redux/middleware/socketMiddleware.js Outdated
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

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.

@Turnlings Turnlings requested review from a team and 1Blademaster November 20, 2025 14:40
Copy link
Copy Markdown
Member

@1Blademaster 1Blademaster left a comment

Choose a reason for hiding this comment

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

Haha yeah that was easy 😁

@Turnlings Turnlings merged commit ac789fe into main Nov 20, 2025
13 checks passed
@Turnlings Turnlings deleted the 860-add-mavlink-forwarding-when-rebooting-drone branch November 20, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add mavlink forwarding when rebooting drone

3 participants