Skip to content
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(react): added input change callback to multiselect filterable #11415

Conversation

TannerS
Copy link
Contributor

@TannerS TannerS commented May 13, 2022

Closes #

N/A

Changelog

New

Added a new callback for users can get the input as it is being typed.

Due to a carbon 10 -> carbon11 update on the filterable component, it broken some ref info for our service in production and this needs to be fixed sooner than later

Testing / Reviewing

Wanting to test in storybook for current action detection but currently having issues with that

@github-actions
Copy link
Contributor

github-actions bot commented May 13, 2022

DCO Assistant Lite bot All contributors have signed the DCO.

@netlify
Copy link

netlify bot commented May 13, 2022

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 80c7351
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/6283c44a7f09c70008d95bc1
😎 Deploy Preview https://deploy-preview-11415--carbon-components-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented May 13, 2022

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit 80c7351
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/6283c44a9a9a470009a5931d
😎 Deploy Preview https://deploy-preview-11415--carbon-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@TannerS
Copy link
Contributor Author

TannerS commented May 13, 2022

I have read the DCO document and I hereby sign the DCO.

@TannerS
Copy link
Contributor Author

TannerS commented May 13, 2022

I have read the DCO document and I hereby sign the DCO.

@TannerS
Copy link
Contributor Author

TannerS commented May 13, 2022

recheck

@TannerS TannerS marked this pull request as ready for review May 13, 2022 20:35
@TannerS TannerS requested review from a team as code owners May 13, 2022 20:35
@TannerS TannerS requested review from tay1orjones and joshblack May 13, 2022 20:35
@kodiakhq kodiakhq bot merged commit 606664e into carbon-design-system:main May 17, 2022
tay1orjones added a commit to tay1orjones/carbon that referenced this pull request Jun 27, 2022
…arbon-design-system#11415)

* feat(react): added input change callback to multiselect filterable

* chore(react): removed story changes for previous version

* chore(project): update snaps

* chore(react): updated handleOnInputValueChange to remove type param

Co-authored-by: Taylor Jones <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
kodiakhq bot added a commit that referenced this pull request Jun 27, 2022
…11690)

* feat(react): added input change callback to multiselect filterable (#11415)

* feat(react): added input change callback to multiselect filterable

* chore(react): removed story changes for previous version

* chore(project): update snaps

* chore(react): updated handleOnInputValueChange to remove type param

Co-authored-by: Taylor Jones <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>

* chore(project): update caniuse-lite, browserslist db

Co-authored-by: Tanner Summers <[email protected]>
Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
kennylam added a commit to kennylam/carbon that referenced this pull request Jul 30, 2024
…ge-cwc-v2.1.2

chore(merge): merge `release-cwc/v2.1.2` into `main`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants