Skip to content

Alpha 0.1.10/592 add rally point functionality to missions#599

Merged
1Blademaster merged 3 commits intorelease-alpha-0.1.10from
alpha-0.1.10/592-add-rally-point-functionality-to-missions
Aug 3, 2025
Merged

Alpha 0.1.10/592 add rally point functionality to missions#599
1Blademaster merged 3 commits intorelease-alpha-0.1.10from
alpha-0.1.10/592-add-rally-point-functionality-to-missions

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

No description provided.

@1Blademaster 1Blademaster requested a review from Copilot August 3, 2025 16:17
@1Blademaster 1Blademaster linked an issue Aug 3, 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 rally point functionality to the missions system, enabling users to create, edit, and delete rally points alongside existing mission and fence capabilities. The changes unify the handling of different mission types (mission, fence, rally) under a common interface while adding rally-specific features.

Key changes:

  • Unified mission item creation, update, and deletion functions to handle all mission types (mission, fence, rally)
  • Added rally point support to the map interface with proper visual indicators and drag functionality
  • Enhanced rally items table with delete functionality and proper column structure

Reviewed Changes

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

File Description
gcs/src/missions.jsx Core logic for unified mission item handling and rally point creation with home position support
gcs/src/components/missions/rallyItemsTableRow.jsx Added delete functionality and proper table structure for rally items
gcs/src/components/missions/rallyItemsTable.jsx Updated table interface to support delete operations
gcs/src/components/missions/missionsMap.jsx Unified map marker handling and added sequence numbers to rally points

Comment thread gcs/src/missions.jsx
Comment thread gcs/src/missions.jsx
Comment thread gcs/src/missions.jsx Outdated
Comment thread gcs/src/missions.jsx
Comment thread gcs/src/missions.jsx
Copy link
Copy Markdown
Contributor

@Kwash67 Kwash67 left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@1Blademaster 1Blademaster merged commit bffd8a5 into release-alpha-0.1.10 Aug 3, 2025
7 checks passed
@1Blademaster 1Blademaster deleted the alpha-0.1.10/592-add-rally-point-functionality-to-missions branch August 3, 2025 16:54
1Blademaster added a commit that referenced this pull request Sep 28, 2025
* Start adding rally functionality to missions page

* Finish adding rally point functionality

* Address copilot comments
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.

Add rally point functionality to missions

3 participants