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: Module instance delete navigation #31140
fix: Module instance delete navigation #31140
Conversation
WalkthroughThe changes involve enhancements and bug fixes in the application's IDE functionality, particularly around JavaScript (JS) actions and module management. Key updates include importing utilities for better IDE type handling, improving the user experience with visual cues through customizable icons, and refining redirection logic after certain actions, such as deleting JS entities. These modifications aim to streamline the development process within the IDE, ensuring users are guided appropriately through the interface and improving the management of JS and query modules. Changes
Assessment against linked issues
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 (
|
…o package-related-delete-navigation-fixes
@@ -46,12 +52,12 @@ export function* handleJSEntityRedirect(deletedId: string) { | |||
const redirectAction = getNextEntityAfterDelete(deletedId, allJsItems); | |||
switch (redirectAction.action) { | |||
case RedirectAction.CREATE: | |||
history.push(jsCollectionAddURL({})); | |||
history.push(jsCollectionAddURL({ pageId })); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dont force page id here. This will eventually handle module and workflow use cases when they start using Editor Pane Segments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is causing pageId not found error while deleting package instance.
…o package-related-delete-navigation-fixes
/ok-to-test tags="@tag.All" |
Whoa, @tag.All spotted in your test suite! 🚀 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/7926163593. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/7926163593. |
This reverts commit 3757f31.
Description
This PR addresses the below issues,
PR fixes following issue(s)
Fixes #30894
Type of change
Testing
How Has This Been Tested?
Test Plan
Issues raised during DP testing
Checklist:
Dev activity
QA activity:
Test Plan Approved
label after Cypress tests were reviewedTest Plan Approved
label after JUnit tests were reviewedSummary by CodeRabbit
New Features
Enhancements