[19.0][MIG] mail_print: Migration to version 19.0 - #244
Open
eduezerouali-tecnativa wants to merge 8 commits into
Open
[19.0][MIG] mail_print: Migration to version 19.0#244eduezerouali-tecnativa wants to merge 8 commits into
eduezerouali-tecnativa wants to merge 8 commits into
Conversation
Member
|
/ocabot migration mail_print |
carlos-lopez-tecnativa
suggested changes
Aug 10, 2026
carlos-lopez-tecnativa
left a comment
carlos-lopez-tecnativa
left a comment
Contributor
There was a problem hiding this comment.
Please check the CI.
| <field name="report_name">mail_print.report_mail_message</field> | ||
| <field name="report_file">mail_print.report_mail_message</field> | ||
| <field name="print_report_name">(object.display_name)</field> | ||
| <field name="print_report_name">(display_name)</field> |
Contributor
There was a problem hiding this comment.
I got this error with this change.
| ) | ||
| self.start_tour("/web", "mail_print.mail_print_tour", login="admin") | ||
|
|
||
| def test_02_mail_note_not_print_tour(self): |
Contributor
There was a problem hiding this comment.
Why did you remove the other test that verifies that the button is not available in the Notes?
eduezerouali-tecnativa
force-pushed
the
19.0-mig-mail_print
branch
5 times, most recently
from
August 12, 2026 08:00
7e7eafd to
1517e07
Compare
…sive is installed.
Before this commit, the option to print messages was only available for messages sent from Odoo. After this commit, it is now possible to print incoming messages as well.
eduezerouali-tecnativa
force-pushed
the
19.0-mig-mail_print
branch
from
August 12, 2026 10:30
1517e07 to
16700e8
Compare
carlos-lopez-tecnativa
suggested changes
Aug 12, 2026
| }, | ||
| { | ||
| content: "Open contact", | ||
| trigger: ".o_list_table td[name='complete_name']:contains('Mail Print')", |
Contributor
There was a problem hiding this comment.
I see that the test fails because it cannot find this element. Please try using this instead:
Suggested change
| trigger: ".o_list_table td[name='complete_name']:contains('Mail Print')", | |
| trigger: ".o_list_table td[name='display_name']:contains('Mail Print')", |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
cc @Tecnativa TT63995
ping @carlos-lopez-tecnativa @pilarvargas-tecnativa
Moved from social repo