-
-
Notifications
You must be signed in to change notification settings - Fork 588
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
[MIG] base_search_mail_content: Migration to 11.0 #282
[MIG] base_search_mail_content: Migration to 11.0 #282
Conversation
</record> | ||
|
||
<record model="ir.ui.view" id="trgm_index_view_tree"> | ||
<!--<record model="ir.ui.view" id="trgm_index_view_tree"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you comment this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's not needed, it's better to remove it. Git will keep it if needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some details.
@@ -0,0 +1,6 @@ | |||
This module depends on the module 'base_search_fuzzy' to ensure that | |||
searches on emails are based on indexes. Please read carefully the | |||
`install instructions <https://github.com/OCA/server-tools/blob/10.0/base_search_fuzzy/README.rst>`_. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if we should link the README now that it is split... Maybe better the odoo apps site? Besides, it's v10 branch.
</record> | ||
|
||
<record model="ir.ui.view" id="trgm_index_view_tree"> | ||
<!--<record model="ir.ui.view" id="trgm_index_view_tree"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's not needed, it's better to remove it. Git will keep it if needed.
@yajo done! Can you recheck? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved when merged dependency PR
8daa918
to
c946c83
Compare
Please @cubells rebase! |
Introduce search of html fields Create indexes Update Readme
c946c83
to
d93a722
Compare
@rafaelbn done! |
d93a722
to
caf6355
Compare
Thanks @cubells ! Please review translation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Functional tested 👍
Please review authors @cubells
caf6355
to
fee986f
Compare
@yajo @pedrobaeza this is ready to be merged. |
Depends on:
cc @Tecnativa