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.

Sign up for GitHub

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

Jump to bottom

[Task]: Remove existing anvil widget selection UX implementation #33131

Closed

riodeuno opened this issue May 2, 2024 · 0 comments · Fixed by #33129

Assignees

Labels

Anvil Pod Anvil team Task

Contributor

riodeuno commented May 2, 2024

SubTasks

With the new widget selection UX, the existing implementation is no longer needed. Let's remove it.

riodeuno added the Task label

riodeuno self-assigned this

riodeuno added Anvil Pod Anvil team labels

riodeuno mentioned this issue

fix: remove existing widget selection implementation #33129

Merged

2 tasks

riodeuno closed this as completed in #33129

riodeuno added a commit that referenced this issue


          fix: remove existing widget selection implementation (#33129)

e54020b

## Description
With the new widget selection UX. This existing implementation is no
longer necessary.
This PR removes it

Fixes: #33131 

## Automation

/ok-to-test tags="@tag.All"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!CAUTION]
> 🔴 🔴 🔴 Some tests have failed.
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/8960841989>
> Commit: 4f9c6d5
> Cypress dashboard: <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=8960841989&attempt=1&selectiontype=test&testsstatus=failed&specsstatus=fail"
target="_blank"> Click here!</a>
> The following are new failures, please fix them before merging the PR:
<ol>
> <li>cypress/e2e/Regression/ServerSide/QueryPane/DSDocs_Spec.ts </ol>
> To know the list of identified flaky tests - <a
href="https://internal.appsmith.com/app/cypress-dashboard/identified-flaky-tests-65890b3c81d7400d08fa9ee3?branch=master"
target="_blank">Refer here</a>

<!-- end of auto-generated comment: Cypress test results  -->





## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [x] No

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment