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

Fix search block input clear button doesn't reset the search (#4837) #5011

Merged
merged 3 commits into from
Jul 26, 2023

Conversation

sneridagh
Copy link
Member

No description provided.

Co-authored-by: ionlizarazu <ilizarazu@codesyntax.com>
Co-authored-by: Mikel Larreategi <mlarreategi@codesyntax.com>
@sneridagh sneridagh added the backport A backport PR to a numbered branch label Jul 21, 2023
@sneridagh sneridagh added this to the 16.x.x milestone Jul 21, 2023
@netlify
Copy link

netlify bot commented Jul 21, 2023

Deploy Preview for volto canceled.

Name Link
🔨 Latest commit b03050b
🔍 Latest deploy log https://app.netlify.com/sites/volto/deploys/64c0d13e842d1a0008015857

@cypress
Copy link

cypress bot commented Jul 21, 2023

Passing run #6610 ↗︎

0 637 25 0 Flakiness 0

Details:

add an accordion click in the cypress tests to open FACETS and CONTROLS
Project: Volto Commit: b03050be30
Status: Passed Duration: 15:17 💡
Started: Jul 26, 2023 7:57 AM Ended: Jul 26, 2023 8:13 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@sneridagh
Copy link
Member Author

@ionlizarazu @iFlameing there are some problems with the backport of this one.
Could be that both search blocks differ or the tests are different.
Is it backport-able?

@ionlizarazu
Copy link
Contributor

@sneridagh I've fixed the tests for this branch. The functionality in the master and search-reset-clear__16 branches is different. In master, when you add a search block, all the settings accordions are open, in this branch there are closed by default.
I think we should keep passing the tests now, and when the functionality changes we should consciously modify the tests as well.

@sneridagh
Copy link
Member Author

@ionlizarazu thanks a lot for taking care! LGTM, merging.

@sneridagh sneridagh merged commit c80d3c7 into 16.x.x Jul 26, 2023
@sneridagh sneridagh deleted the search-reset-clear__16 branch July 26, 2023 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport A backport PR to a numbered branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants