Skip to content

Add clear mission button to context menu#603

Merged
NexInfinite merged 4 commits intorelease-alpha-0.1.10from
alpha-0.1.10/602-add-clear-mission-button-on-missions-context-menu
Aug 6, 2025
Merged

Add clear mission button to context menu#603
NexInfinite merged 4 commits intorelease-alpha-0.1.10from
alpha-0.1.10/602-add-clear-mission-button-on-missions-context-menu

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

No description provided.

@1Blademaster 1Blademaster requested a review from Copilot August 3, 2025 18:04
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 a "Clear mission" button to the context menu on the missions map that allows users to clear mission items for different mission types (mission, fence, rally).

Key changes:

  • Backend logic updated to handle empty waypoint uploads for fence and rally mission types
  • Frontend context menu enhanced with clear mission functionality
  • Clear mission function handles different mission types appropriately

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
radio/app/controllers/missionController.py Updates upload logic to handle clearing fence and rally missions when no waypoints are provided
gcs/src/missions.jsx Adds clearMissionItems function that clears items based on active tab type
gcs/src/components/missions/missionsMap.jsx Adds "Clear mission" context menu item that calls the clear function

Comment thread radio/app/controllers/missionController.py Outdated
Comment thread gcs/src/components/missions/missionsMap.jsx
@1Blademaster 1Blademaster force-pushed the alpha-0.1.10/602-add-clear-mission-button-on-missions-context-menu branch from 3da6d77 to 4946665 Compare August 3, 2025 18:24
Copy link
Copy Markdown
Contributor

@NexInfinite NexInfinite left a comment

Choose a reason for hiding this comment

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

Works on windows, looks good to me.

@NexInfinite NexInfinite merged commit 8c4e9d1 into release-alpha-0.1.10 Aug 6, 2025
7 checks passed
@NexInfinite NexInfinite deleted the alpha-0.1.10/602-add-clear-mission-button-on-missions-context-menu branch August 6, 2025 11:51
1Blademaster added a commit that referenced this pull request Sep 28, 2025
* Add clear mission button to context menu

* Clear mission for all mission types

* Fix no waypoints mission test

* Fix test again
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.

3 participants