-
-
Notifications
You must be signed in to change notification settings - Fork 166
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] event_mail: Migration to version 16.0 #316
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/
f99aad1
to
ba3f972
Compare
@@ -9,6 +9,7 @@ | |||
<xpath expr="//field[@name='event_mail_ids']" position="before"> | |||
<field | |||
name="event_mail_template_id" | |||
groups="base.group_user" |
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 is this 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.
When trying to install the module, it gave me an error complaining that
Field 'seats_expected' used in attrs ({'readonly': [('seats_expected', '>', 0)]}) is restricted to the group(s) base.group_user.
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.
Ok, thanks
/ocabot migration event_mail |
What a great day to merge this nice PR. Let's do it! |
It looks like something changed on |
Congratulations, your PR was merged at 838e13c. Thanks a lot for contributing to OCA. ❤️ |
cc @Tecnativa TT43665
@ernestotejeda @chienandalu please review!