feat(HappyHare): Added to gate context menu for complete change-tool operation#2430
feat(HappyHare): Added to gate context menu for complete change-tool operation#2430moggieuk wants to merge 2 commits into
Conversation
📝 WalkthroughWalkthroughAdded a new "Change Tool" context-menu button to the MMU unit gate component, exposing a new public Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/components/panels/Mmu/MmuUnitGate.vue`:
- Around line 104-108: The MMU_CHANGE_TOOL action is sending a GATE= parameter
but should send TOOL=<index>; update the gateCommand method (or add a new
toolCommand helper) used by the MMU_CHANGE_TOOL menu entry so it emits
"MMU_CHANGE_TOOL TOOL=<toolIndex>" instead of "GATE=<...>"; either map the
current gateIndex to the correct tool index before dispatching or implement a
separate method (e.g., sendMmuToolChange) that builds the "MMU_CHANGE_TOOL
TOOL=" payload and use that for the MMU_CHANGE_TOOL command in the menu
definition.
|
Closed and replaced with PR #2437 |
Description
This PR simply adds an additional action button to the per-gate pop-up menu to request a complete toolchange. This is for convenience and potentially saves the user time with separate UNLOAD / SELECT / LOAD operations.
Related Tickets & Documents
n/a
Mobile & Desktop Screenshots/Recordings
Simple adds the "Change Tool" menu item:

[optional] Are there any post-deployment tasks we need to perform?
n/a