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

[17.0][MIG] web_search_with_and #3090

Open
wants to merge 29 commits into
base: 17.0
Choose a base branch
from

Conversation

JasminSForgeFlow
Copy link
Contributor

Standard Migration

@ForgeFlow

Andrius Preimantas and others added 29 commits February 12, 2025 11:19
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/
@JasminSForgeFlow JasminSForgeFlow marked this pull request as ready for review February 12, 2025 08:38
Copy link

@JordiMForgeFlow JordiMForgeFlow left a comment

Choose a reason for hiding this comment

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

code + functional review

Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional review. It works.

Copy link

@Jortolsa-S73 Jortolsa-S73 left a comment

Choose a reason for hiding this comment

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

Functional review

@LoisRForgeFlow
Copy link
Contributor

This PR is superseding #2876. I hope it now include a fix for the concerns @StefanRijnhart had there. @StefanRijnhart would you mind taking a look?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.