-
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: select widget search improvement #17006
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
/ok-to-test=2afda63 |
Tested this PR and working as expected |
…lect-widget-search
Deployment failed with the following error:
|
/ok-to-test sha=3e9ea44 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3157579562. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/3157579562. Click to view performance test results
|
@souma-ghosh Could you please fix the CI, Will approve post that |
…lect-widget-search
/ok-to-test sha=41e1f22 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3173043148. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/3173043148. Click to view performance test results
|
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3304974773. |
/ok-to-test sha=2afda63 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3304995588. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3305036734. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3305037049. |
Description
Updated the
DropdownOption
to allow the label to be either number/string.I left the
EXPECTED STRUCTURE
as a string because numbers shouldn't be supported by default.Also added a cypress test to ensure filtering works and
Select.SelectedOptionLabel
outputs the correct typeFixes #17003
Type of change
How Has This Been Tested?
Checklist: