Skip to content

Add gripper related params on config page#744

Merged
1Blademaster merged 5 commits intomainfrom
743-add-gripper-configuration-to-gripper-page
Oct 4, 2025
Merged

Add gripper related params on config page#744
1Blademaster merged 5 commits intomainfrom
743-add-gripper-configuration-to-gripper-page

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

@1Blademaster 1Blademaster commented Oct 3, 2025

image

@1Blademaster 1Blademaster requested a review from Copilot October 3, 2025 20:19
@1Blademaster 1Blademaster linked an issue Oct 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 gripper configuration parameters to the config page, enabling users to view and modify gripper-related settings through the web interface.

  • Adds new backend endpoints to get and set gripper configuration parameters
  • Updates the gripper controller to handle parameter retrieval and updates
  • Adds Redux state management for gripper config data
  • Implements a comprehensive frontend interface for gripper parameter configuration

Reviewed Changes

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

Show a summary per file
File Description
radio/app/endpoints/gripper.py Adds socket endpoints for getting and setting gripper configuration
radio/app/controllers/gripperController.py Refactors parameter handling and adds config management methods
gcs/src/redux/slices/configSlice.js Adds Redux state and actions for gripper configuration
gcs/src/redux/middleware/socketMiddleware.js Adds socket event handlers for gripper config
gcs/src/redux/middleware/emitters.js Adds socket emitters for gripper config operations
gcs/src/components/config/gripper.jsx Implements comprehensive UI for gripper parameter configuration

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

Comment thread radio/app/endpoints/gripper.py Outdated
Comment thread radio/app/controllers/gripperController.py Outdated
Comment thread gcs/src/redux/slices/configSlice.js Outdated
Comment thread gcs/src/redux/slices/configSlice.js Outdated
Comment thread gcs/src/redux/slices/configSlice.js Outdated
Comment thread gcs/src/redux/slices/configSlice.js Outdated
@1Blademaster 1Blademaster merged commit c75a32a into main Oct 4, 2025
8 checks passed
@1Blademaster 1Blademaster deleted the 743-add-gripper-configuration-to-gripper-page branch October 4, 2025 11:46
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.

Add gripper configuration to gripper page

3 participants