-
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
feat: remote js execution code split #32853
Conversation
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: 3
app/client/src/components/editorComponents/JSResponseView.test.tsx
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,3 @@ | |||
export default function JSRemoteExecutionView({}: any) { |
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.
I think you have missed the CE version of this 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.
Updated, please check
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.
LGTM
Description
This PR is a code split PR for EE feature which allows remote JS execution for JS functions. The changes majorly relate to workflows code. Changelog:
JsPaneSagas
Fixes #32318
Fixes #32320
Fixes #32321
Automation
/ok-to-test tags="@tag.Sanity, @tag.JS"
🔍 Cypress test results
Tip
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/8946098385
Commit: ff7aeec
Cypress dashboard url: Click here!
Communication
Should the DevRel and Marketing teams inform users about this change?