Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: custom command in context menu #1123

Merged
merged 2 commits into from
Sep 20, 2023
Merged

fix: custom command in context menu #1123

merged 2 commits into from
Sep 20, 2023

Conversation

abeatrix
Copy link
Contributor

Issue

Currently, none of the commands from the Custom Commands menu work.

Steps to reproduce:

  1. Open the Custom Command menu from the right click menu in your editor
    image
  2. Choose a Custom Command to run from the menu
  3. None of the commands would work

This PR includes:

  • Fixed the issue mentioned above
  • added a back button to move from Custom Command Menu to Cody Menu

image

Also made the following changes in our package.json

  • Rename "Smell Code" to "Find Code Smells" for clarity
  • Move commands to "Ask Cody" category for better organization
  • Disable duplicated commands in command palette as /doc /test /smell etc are already covered in our Commands Menu
  • Organize Cody submenu into "command" and "other" groups
  • Rename "Start New Thread" to "Ask Cody Inline"
  • Rename "Custom Commands (Experimental)" to just "Custom Commands"
  • Rename "Fixup" command to "Edit Code"
BEFORE AFTER
image image

Test plan

  1. Start Cody from this branch
  2. Open the Custom Command menu from the right-click menu in your editor
    image
  3. Choose a Custom Command to run from the menu
  4. Command should work as expected

@abeatrix abeatrix requested review from taras-yemets and a team September 20, 2023 13:58
@abeatrix abeatrix enabled auto-merge (squash) September 20, 2023 14:06
@abeatrix abeatrix merged commit 0c4d320 into main Sep 20, 2023
10 checks passed
@abeatrix abeatrix deleted the bee/fix-custom-menu branch September 20, 2023 14:06
@abeatrix abeatrix self-assigned this Sep 20, 2023
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.

None yet

2 participants