-
-
Notifications
You must be signed in to change notification settings - Fork 160
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
[15.0][MIG] event_mail: Migration to version 15.0 #312
Conversation
Currently translated at 100.0% (22 of 22 strings) Translation: event-10.0/event-10.0-event_mail Translate-URL: https://translation.odoo-community.org/projects/event-10-0/event-10-0-event_mail/fr/
Currently translated at 100.0% (22 of 22 strings) Translation: event-10.0/event-10.0-event_mail Translate-URL: https://translation.odoo-community.org/projects/event-10-0/event-10-0-event_mail/es/
Currently translated at 76.9% (30 of 39 strings) Translation: event-14.0/event-14.0-event_mail Translate-URL: https://translation.odoo-community.org/projects/event-14-0/event-14-0-event_mail/it/
event_mail/models/event_mail.py
Outdated
"template_ref": "mail.template, %i" | ||
% self.env.ref("event.event_reminder").id, |
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.
"template_ref": "mail.template, %i" | |
% self.env.ref("event.event_reminder").id, | |
"template_ref": f"mail.template, {self.env.ref("event.event_reminder").id}", |
9647427
to
319325a
Compare
@chienandalu could you review again? |
please review @stefan-tecnativa |
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.
Functionally tested, LGTM. Just check the comments.
319325a
to
615cd13
Compare
@stefan-tecnativa changes done |
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.
Thanks everyone :)
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.
LG
/ocabot migration event_mail |
Hey, thanks for contributing! Proceeding to merge this for you. |
Congratulations, your PR was merged at 565517a. Thanks a lot for contributing to OCA. ❤️ |
Cc @Tecnativa TT42036