Skip to content

Convert config to redux#719

Merged
NexInfinite merged 15 commits intorelease-alpha-0.1.10from
alpha-0.1.10/658-convert-config-to-redux
Sep 26, 2025
Merged

Convert config to redux#719
NexInfinite merged 15 commits intorelease-alpha-0.1.10from
alpha-0.1.10/658-convert-config-to-redux

Conversation

@NexInfinite
Copy link
Copy Markdown
Contributor

No description provided.

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 converts configuration-related state management from direct socket handling to Redux state management by introducing a new configSlice and updating all related components.

  • Creates a centralized config Redux slice to manage gripper, flight modes, frame configuration, and radio channel data
  • Updates all config components to use Redux hooks instead of direct socket communication
  • Migrates socket event handling to the Redux middleware pattern

Reviewed Changes

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

Show a summary per file
File Description
gcs/src/redux/store.js Adds configSlice to the root reducer
gcs/src/redux/slices/droneInfoSlice.js Adds gripperEnabled state field
gcs/src/redux/slices/configSlice.js New Redux slice for configuration state management
gcs/src/redux/middleware/socketMiddleware.js Adds config-specific socket event handlers and RC_CHANNELS processing
gcs/src/redux/middleware/emitters.js Adds config-related socket emission handlers
gcs/src/config.jsx Converts to use Redux for gripper state instead of direct socket
gcs/src/components/config/radioCalibration.jsx Migrates from local state to Redux for radio channels
gcs/src/components/config/motorTest.jsx Converts motor test functionality to Redux
gcs/src/components/config/gripper.jsx Updates gripper component to use Redux dispatch
gcs/src/components/config/flightModes.jsx Migrates flight modes to Redux state management
.vscode/settings.json Adds spelling dictionary entries for new terms

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

Comment thread gcs/src/redux/middleware/socketMiddleware.js Outdated
Comment thread gcs/src/redux/slices/configSlice.js Outdated
Comment thread gcs/src/components/config/motorTest.jsx Outdated
Comment thread gcs/src/redux/middleware/socketMiddleware.js
@NexInfinite NexInfinite changed the title Alpha 0.1.10/658 convert config to redux Convert config to redux Sep 26, 2025
@NexInfinite NexInfinite merged commit 379a1a6 into release-alpha-0.1.10 Sep 26, 2025
7 checks passed
@NexInfinite NexInfinite deleted the alpha-0.1.10/658-convert-config-to-redux branch September 26, 2025 20:54
1Blademaster pushed a commit that referenced this pull request Sep 28, 2025
* initial config changes

* moved to own slice

* flightMode.jsx done

* gripper.jsx done

* motorTest.jsx done

* radioCalibration.jsx done

* formatted

* fixed rc and gripper pages

* fixed updating frame type and class

* fixed wording

* gripperenabled

* formatted

* fixed tests and moved to is_gripper_enabled

* updated flight modes on load

* formatted
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.

3 participants