Skip to content

Add force arm functionality#1089

Merged
1Blademaster merged 3 commits intomainfrom
1072-feature-be-able-to-force-arm
Mar 16, 2026
Merged

Add force arm functionality#1089
1Blademaster merged 3 commits intomainfrom
1072-feature-be-able-to-force-arm

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings March 16, 2026 20:45
@1Blademaster 1Blademaster linked an issue Mar 16, 2026 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

Adds “force arm” UX in the GCS so that when a normal arm attempt fails, the user can be prompted to retry with MAVLink force arming enabled (mirroring the existing force-disarm flow).

Changes:

  • Add forceArmModalOpened state + reducer/selector wiring in droneConnectionSlice.
  • Open a new Force Arm modal from socketMiddleware when a non-force arm attempt fails.
  • Introduce ForceArmModal component and render it in the main Dashboard.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
radio/app/controllers/armController.py Adds arm/disarm logging for force vs non-force attempts.
gcs/src/redux/slices/droneConnectionSlice.js Adds Redux state/actions/selectors for Force Arm modal visibility.
gcs/src/redux/middleware/socketMiddleware.js Triggers Force Arm modal on failed non-force arming responses.
gcs/src/dashboard.jsx Renders the new Force Arm modal alongside existing modals.
gcs/src/components/dashboard/tabsSectionTabs/actionTabsSection.jsx Simplifies arm/disarm dispatch to explicitly pass force: false.
gcs/src/components/dashboard/ForceDisarmModal.jsx Updates copy/props for the existing Force Disarm modal.
gcs/src/components/dashboard/ForceArmModal.jsx New modal that dispatches emitArmDisarm({ arm: true, force: true }).

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

Comment thread gcs/src/redux/middleware/socketMiddleware.js Outdated
@1Blademaster 1Blademaster merged commit 80c702d into main Mar 16, 2026
8 checks passed
@1Blademaster 1Blademaster deleted the 1072-feature-be-able-to-force-arm branch March 16, 2026 21:09
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.

[FEATURE] Be able to force arm

2 participants