Skip to content
This repository has been archived by the owner on Dec 30, 2022. It is now read-only.

fix(hooks-web): reset with searchAsYouType=false should search #3642

Merged
merged 2 commits into from
Sep 29, 2022

Conversation

Haroenv
Copy link
Contributor

@Haroenv Haroenv commented Sep 28, 2022

Summary

Similarly as #802, when reset is triggered and searchAsYouType is false, a search should still happen.

Result

fixes #3641

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 28, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 4eb6d6b:

Sandbox Source
react-instantsearch-app Configuration
hooks-example Configuration
magical-alex-xizl3l Issue #3641

@netlify
Copy link

netlify bot commented Sep 28, 2022

Deploy Preview for react-instantsearch ready!

Name Link
🔨 Latest commit 4eb6d6b
🔍 Latest deploy log https://app.netlify.com/sites/react-instantsearch/deploys/63357ed35f3b5e000814fae1
😎 Deploy Preview https://deploy-preview-3642--react-instantsearch.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.

Copy link
Member

@aymeric-giraudet aymeric-giraudet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We got fooled twice 😅

@Haroenv Haroenv enabled auto-merge (squash) September 29, 2022 11:18
@Haroenv Haroenv merged commit f969deb into master Sep 29, 2022
@Haroenv Haroenv deleted the fix/reset-searchAsYouType branch September 29, 2022 11:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Query does clear on reset when using searchAsYouType set to false in the <SearchBox> component
2 participants