Skip to content

Adding Redux base files#614

Merged
NexInfinite merged 4 commits intorelease-alpha-0.1.10from
alpha-0.1.10/613-redux-base-files
Aug 17, 2025
Merged

Adding Redux base files#614
NexInfinite merged 4 commits intorelease-alpha-0.1.10from
alpha-0.1.10/613-redux-base-files

Conversation

@NexInfinite
Copy link
Copy Markdown
Contributor

No description provided.

@NexInfinite NexInfinite linked an issue Aug 17, 2025 that may be closed by this pull request
@NexInfinite NexInfinite changed the base branch from main to release-alpha-0.1.10 August 17, 2025 14:29
@NexInfinite NexInfinite requested a review from Copilot August 17, 2025 14:29
Comment thread gcs/src/redux/middleware/socketMiddleware.js
Comment thread gcs/src/redux/slices/droneInfoSlice.js

This comment was marked as outdated.

@1Blademaster 1Blademaster requested a review from Copilot August 17, 2025 14:58
@NexInfinite NexInfinite requested review from Copilot and removed request for Copilot August 17, 2025 14:59
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 introduces Redux base files to establish state management infrastructure for a Ground Control Station (GCS) application. The changes set up multiple Redux slices for managing different aspects of drone operations and add middleware for socket communication.

  • Sets up Redux store configuration with multiple slices for drone operations, socket connections, and notifications
  • Adds Redux middleware for handling socket communication and message dispatching
  • Introduces base Redux slices for managing drone information, mission data, status messages, and connection states

Reviewed Changes

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

Show a summary per file
File Description
gcs/src/redux/store.js Configures Redux store with combined slices and socket middleware
gcs/src/redux/middleware/socketMiddleware.js Implements Redux middleware for socket communication and message handling
gcs/src/redux/slices/*.js Define Redux slices for drone info, connections, missions, notifications, and status
gcs/src/redux/subscribers/localStorageSubscriber.js Placeholder file for local storage persistence (currently commented out)
gcs/src/helpers/socket.js Adds socket factory pattern for managing socket connections
Other files Minor formatting and style improvements

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread gcs/src/redux/slices/notificationSlice.js Outdated
Comment thread gcs/src/redux/slices/notificationSlice.js Outdated
Comment thread gcs/src/redux/slices/socketSlice.js Outdated
Comment thread gcs/src/redux/slices/droneInfoSlice.js
Comment thread gcs/src/redux/slices/droneInfoSlice.js Outdated
Comment thread gcs/src/redux/slices/droneConnectionSlice.js Outdated
Comment thread gcs/src/redux/middleware/socketMiddleware.js Outdated
Comment thread gcs/src/redux/middleware/socketMiddleware.js Outdated
Comment thread gcs/src/redux/middleware/socketMiddleware.js Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

LGTM

@NexInfinite NexInfinite merged commit d51c43c into release-alpha-0.1.10 Aug 17, 2025
7 checks passed
@NexInfinite NexInfinite deleted the alpha-0.1.10/613-redux-base-files branch August 17, 2025 15:12
1Blademaster pushed a commit that referenced this pull request Sep 28, 2025
* added base files

* fixed format

* Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fixed copilots error

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

Redux base files

3 participants