Skip to content

Conversation

@lef-adhoc
Copy link
Contributor

No description provided.

Andrius Preimantas and others added 30 commits November 6, 2025 11:34
Add README.rst

Too short underline for module title in README.rst

Improving module meta information

Version 1.0

W391 blank line at end of file

Remove module description because README.rst is there

web_search_with_and: Latest OCA conventions
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: web-16.0/web-16.0-web_search_with_and
Translate-URL: https://translation.odoo-community.org/projects/web-16-0/web-16-0-web_search_with_and/
@lef-adhoc lef-adhoc mentioned this pull request Nov 6, 2025
42 tasks
@Quan-nhm
Copy link

Quan-nhm commented Nov 7, 2025

LGTM, Functional Test OK

Copy link

@lav-adhoc lav-adhoc left a comment

Choose a reason for hiding this comment

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

LGTM :)

@Vang-NguyenPhu
Copy link
Contributor

Hi, really appreciate your migration PR. After testing, the feature works great, only one minor thing I think you need to re-check.

The input dropdown does not totally disappear after I press Enter.

image

I think you can put this line of code to close the input dropdown. I'm not sure if it's the right way to handle it. But you can take it into account.

image

@lef-adhoc lef-adhoc force-pushed the 19.0-mig-web_search_with_and branch from b671ccb to 90a7413 Compare November 21, 2025 15:18
@lef-adhoc
Copy link
Contributor Author

@Vang-NguyenPhu Thank you very much, that's all sorted!

Copy link
Contributor

@Vang-NguyenPhu Vang-NguyenPhu left a comment

Choose a reason for hiding this comment

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

Functional test & code LGTM

@OCA-git-bot
Copy link
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). 🤖

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.