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
chore: Re-ordering the menu for APIs and Queries & deleting BetaTag component as its deprecated #31552
Conversation
WalkthroughWalkthroughThe changes made across several files in the editor's explorer module primarily focused on refining the user interface and enhancing the functionality of context menus. Specifically, these modifications include reordering context menu items for a better user experience, updating the type of labels in dropdown options to support React nodes, and adding a new context menu item for creating modules from existing queries. Changes
Assessment against linked issues
By analyzing the changes made, the objectives outlined in the linked issue have been addressed. The order of the "Create module" option in the context menu now aligns with the design specifications, as it has been explicitly added and the logic around menu item inclusion has been refined to ensure correct ordering and functionality. Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
/ok-to-test tags="@tag.All" |
Whoa, @tag.All spotted in your test suite! 🚀 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/8170294250. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8170294250.
To know the list of identified flaky tests - Refer here |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8170294250. |
Description
BetaTag
component as its deprecated and not being used anymoreTreeDropdownOption
interface to support EE changesPR fixes following issue(s)
Fixes #31319 #32518
Type of change
Testing
How Has This Been Tested?
Checklist:
Dev activity
QA activity:
Test Plan Approved
label after Cypress tests were reviewedTest Plan Approved
label after JUnit tests were reviewedSummary by CodeRabbit