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

16.0[MIG] base_search_mail_content #1164

Merged
merged 26 commits into from
Jun 14, 2023

Conversation

anddago78
Copy link

This module adds the capability to find on any object (e.g. project issues or
helpdesk ticket) based on the conversation threads associated to them.

MT-3086 @moduon @rafaelbn @yajo @Shide please reviews :)

serpentcs-dev1 and others added 25 commits June 12, 2023 09:46
Introduce search of html fields
Create indexes
Update Readme
On views with a searchpanel, xpath `//field[last()]` may select an element inside
the search panel, provoking the error "Only types ['many2one'] are supported for
category (found type text)".

This fix uses an stricter xpath for avoiding this situation.

TT25796
Avoid CacheMiss errors setting a value on message_content field.
Currently translated at 100.0% (9 of 9 strings)

Translation: social-15.0/social-15.0-base_search_mail_content
Translate-URL: https://translation.odoo-community.org/projects/social-15-0/social-15-0-base_search_mail_content/ca/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: social-15.0/social-15.0-base_search_mail_content
Translate-URL: https://translation.odoo-community.org/projects/social-15-0/social-15-0-base_search_mail_content/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: social-15.0/social-15.0-base_search_mail_content
Translate-URL: https://translation.odoo-community.org/projects/social-15-0/social-15-0-base_search_mail_content/
@anddago78 anddago78 changed the title 16.0 mig base search mail content 16.0[MIG] base_search_mail_content Jun 12, 2023
Copy link
Contributor

@Shide Shide left a comment

Choose a reason for hiding this comment

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

I'm not able to search on message_content field (the one that adds this module) on any view.

The option to search should appear on the bottom of the search filters.

mc

@anddago78 anddago78 marked this pull request as draft June 13, 2023 11:58
@anddago78 anddago78 force-pushed the 16.0-mig-base_search_mail_content branch from 0e66448 to e5dec4b Compare June 13, 2023 13:06
@anddago78
Copy link
Author

HI @Shide there was a fields_view_get function that has been deprecated in the version 16 , i have changed this function with another function and now work correctly, thanks for your comments...

@anddago78 anddago78 marked this pull request as ready for review June 13, 2023 13:28
@rafaelbn
Copy link
Member

/ocabot migration base_search_mail_content

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Jun 13, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 13, 2023
38 tasks
@pedrobaeza
Copy link
Member

You should check #1154

Copy link
Contributor

@Shide Shide left a comment

Choose a reason for hiding this comment

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

Tested and works fine

Copy link
Member

@yajo yajo left a comment

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-1164-by-yajo-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 4b07331 into OCA:16.0 Jun 14, 2023
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 1bcbcd3. Thanks a lot for contributing to OCA. ❤️

@yajo yajo deleted the 16.0-mig-base_search_mail_content branch June 14, 2023 06:45
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.