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: ds discard popup issue fixes #19114
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
/ok-to-test sha=634bca8 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3750304565. |
/ok-to-test sha=4adba5f |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3752388715. |
app/client/cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/DSDiscardBugs_spec.ts
Outdated
Show resolved
Hide resolved
app/client/cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/DSDiscardBugs_spec.ts
Outdated
Show resolved
Hide resolved
app/client/cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/DSDiscardBugs_spec.ts
Outdated
Show resolved
Hide resolved
app/client/cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/DSDiscardBugs_spec.ts
Outdated
Show resolved
Hide resolved
app/client/cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/DSDiscardBugs_spec.ts
Outdated
Show resolved
Hide resolved
@Aishwarya-U-R Resolved all code review comments. Please check |
/ok-to-test sha=bf95f10 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3757524651. |
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.
Cypress looks Good!
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/DSDiscardBugs_spec.ts |
/ok-to-test sha=e5701e5 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3785753733. |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/Binding/xmlParser_spec.js |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/Autocomplete_JS_spec.ts |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/Autocomplete_JS_spec.ts |
2 similar comments
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/Autocomplete_JS_spec.ts |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/BugTests/Autocomplete_JS_spec.ts |
/ok-to-test sha=e5701e5 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3791995848. |
The following are new failures, please fix them before merging the PR cypress/integration/Smoke_TestSuite/ClientSideTests/Templates/Fork_Template_Existing_app_spec.js |
/perf-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3800921580. |
Description
This PR includes following changes:
TL;DR
Fixes #18962 , #18998
Media
Screen.Recording.2022-12-21.at.8.09.38.PM.mov
Type of change
How Has This Been Tested?
Test Plan
Issues raised during DP testing
Checklist:
Dev activity
QA activity: