[IMP] mail_full_expand: Migrated to 10.0 #112
Conversation
bouvyd
changed the title
| @@ -5,13 +5,13 @@ | |||
| { | |||
| "name": "Mail full expand", | |||
| "summary": "Expand mail in a big window", | |||
| "version": "8.0.3.0.0", | |||
| "version": "10.0.3.0.0", | |||
There was a problem hiding this comment.
Are you sure you did not mean 10.0.1.0.01.1.1alpha+e? 😛
| <field name="notified_partner_ids" | ||
| widget="many2many_tags" | ||
| <odoo> | ||
| <record id="view" model="ir.ui.view"> |
There was a problem hiding this comment.
I recommend we switch this id to match OCA naming standards: mail_message_view_form. Non-blocking
|
|
||
| </data> | ||
| </openerp> | ||
| <record id="act_window" model="ir.actions.act_window"> |
There was a problem hiding this comment.
I recommend we switch this id to match OCA naming standards: mail_message_action. Non-blocking
| <record id="view" model="ir.ui.view"> | ||
| <field name="name">mail.message.full_expand</field> | ||
| <field name="model">mail.message</field> | ||
| <field name="priority">30</field> |
There was a problem hiding this comment.
Yes, otherwise the action on the "mail messages" menuitem in technical settings will load this view instead of the "standard" one (mail.view_message_form) which has, for some unfathomable reason, a priority of 20 instead of the 16 default...
|
@dbo-odoo Are you going to work on this? |
|
I'm not dead :-) Sorry, I didn't have a lot of free time. I'll finish the current PRs when possible with a bit less productivity than last time :p |
|
hahaha, OK, good to see you around again! |
There was a problem hiding this comment.
Thanks for the updates @dbo-odoo - looks like we have a CI 🔴 though 😦 https://travis-ci.org/OCA/social/jobs/178887566#L561
|
@lasley Up! :-) (like an idiot I applied your xmlid changes without grepping them first O_o) |
No description provided.