Skip to content

[19.0][ADD] web_search_deactivate_autosearch - #3643

Open
samirGuesmi wants to merge 1 commit into
OCA:19.0from
acsone:19.0-add-web_search_deactivate_autosearch-sgu
Open

samirGuesmi wants to merge 1 commit into
OCA:19.0from
acsone:19.0-add-web_search_deactivate_autosearch-sgu

Conversation

@samirGuesmi

Copy link
Copy Markdown
Member

By default, Odoo triggers a new search every time you add or remove a filter, group by, or favorite in the search bar.

This module adds an "Auto search" checkbox next to the search bar: when unchecked, you can combine several filters without triggering a search on each change, and only run the search once by clicking the search icon.

@OCA-git-bot OCA-git-bot added series:19.0 mod:web_search_deactivate_autosearch Module web_search_deactivate_autosearch labels Aug 19, 2026

@AnizR AnizR left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code LGTM
And it seems to work correctly on runboat 👍

@petrus-v petrus-v left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

"author": "ACSONE SA/NV, Odoo Community Association (OCA)",
"license": "AGPL-3",
"installable": True,
"depends": ["web"],

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@samirGuesmi Thanks for the contribution!

Could you also add yourself as a maintainer in the manifest? It helps ensure that modules don’t become orphaned and makes it clear who is taking care of them.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@samirGuesmi
samirGuesmi force-pushed the 19.0-add-web_search_deactivate_autosearch-sgu branch from e41d853 to 1c1f609 Compare September 4, 2026 13:23

@BhaveshHeliconia BhaveshHeliconia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@ivs-cetmix

Copy link
Copy Markdown
Member

Hi @lmignon could you please check if your comment is resolved so we could have it merged?

@lmignon lmignon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM: Code + Functional

@ivs-cetmix

Copy link
Copy Markdown
Member

Hey @samirGuesmi , thank you for your contribution!
/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 19.0-ocabot-merge-pr-3643-by-ivs-cetmix-bump-nobump, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

@ivs-cetmix The merge process could not be finalized, because command twine upload --disable-progress-bar --non-interactive --repository-url https://upload.pypi.org/legacy/ -u __token__ odoo_addon_web_search_deactivate_autosearch-19.0.1.0.0.2-py3-none-any.whl failed with output:

Uploading distributions to https://upload.pypi.org/legacy/
Uploading 
odoo_addon_web_search_deactivate_autosearch-19.0.1.0.0.2-py3-none-any.whl
�[33mWARNING �[0m Error during upload. Retry with the --verbose option for more details. 
�[31mERROR   �[0m HTTPError: 429 Too Many Requests from https://upload.pypi.org/legacy/  
         Too Many Requests                                                      

@lmignon

lmignon commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

@sbidoul Can you reserve the name on pypi plz? Thank you.

@sbidoul

sbidoul commented Sep 15, 2026

Copy link
Copy Markdown
Member

name reserved

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.

8 participants