-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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: Fixes CREATE_JS_ACTION_ERROR error that disallows creating js objects or updating actions on certain git connected apps #33303
Conversation
WalkthroughWalkthroughThe recent updates focus on enhancing the management of actions and layouts within branched environments. Key changes include the introduction of generic type declarations, improved retrieval and handling of branch-specific details in actions, and refined methods for updating action collections based on branch contexts. These modifications aim to streamline operations and ensure consistency across different versions and branches of projects. Changes
Recent Review DetailsConfiguration used: CodeRabbit UI Files selected for processing (3)
Additional comments not posted (3)
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 (
|
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.
Actionable comments posted: 2
Description
When a user develops on a feature branch, merges to master, and then deletes the original branch, the "default entity" associated to all the entities created in the feature branch would now be pointing to a resource that has been deleted on the database. These entities could be NewPage, NewAction, ActionCollection, or ModuleInstances.
The issue that this PR fixes had to do with the fact that any create or update operation on collections and actions respectively would attempt to first retrieve one such deleted resource that was the default entity wrt the current branch and failures in this operation would not even cause exceptions in the flow but simply return to the client side with 2xx responses.
It is possible that other operations of this nature might exist through the code base, but we'll pick them up on an as needed basis since we're hoping to remove all need for such manipulation with the default resources tech debt resolution.
Fixes #30868
Automation
/ok-to-test tags="@tag.Git"
🔍 Cypress test results
Tip
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9007055258
Commit: 57b3311
Cypress dashboard url: Click here!
Communication
Should the DevRel and Marketing teams inform users about this change?