-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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: Handle data update for query execution failure #28841
Conversation
/ok-to-test |
/build-deploy-preview skip-tests=true |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/6861498106. |
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/6861501653. |
Deploy-Preview-URL: https://ce-28841.dp.appsmith.com |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6861498106.
|
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6861498106.
|
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/6870065527. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6870065527.
|
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6870065527.
|
1a8671f
to
5a34ac3
Compare
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/6880540521. |
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/6880554167. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6880554167.
|
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6880540521.
|
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6880540521.
|
/build-deploy-preview skip-tests=true |
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/6886839178. |
Deploy-Preview-URL: https://ce-28841.dp.appsmith.com |
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/6887174826. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6887174826.
|
898aef8
to
1f5b2e8
Compare
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/6893718814. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6893718814.
|
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/6893718814. |
Description
Sets query data to
undefined
when execution failsPR fixes following issue(s)
Fixes #28817
Media
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 reviewed