-
-
Notifications
You must be signed in to change notification settings - Fork 822
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
[9.0] [MIG] partner_contact_lang #356
Conversation
=========================== Manage language in contacts =========================== Odoo by default propagate language field to the created contacts from their form, but it doesn't allow to change it once created. This module fills this gap, and also provides other facilities for the contact language management: * Put the language of the parent company when the contact doesn't have a language and this parent company is assigned. * When the company changes the language, it fills with the same language all the contacts that don't have any. Usage ===== Go to any partner that is a company and has contacts. Click on one contact and you will be able to edit the language.
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.
Can you change icon to OCA ?
@rousseldenis thanks for your reviews. The icon is a custom one I prepared for the module. I think it's more convenient than the generic one 😉 |
@pedrobaeza I thought it was the convention to use OCA icon in every module |
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.
👍 Code review + tested on runbot
No, the only convention is to not put any commercial trademark or logo, but you can:
I always try to put some icons when I have the inspiration for a clear concept, hehe. Also with the raise of the mobile interface (web_responsive OCA module or web_enterprise module), the icon for the main app is starting to be important. For example, I plan to create an icon for connector, queue and custom info apps. |
@cubells, please review |
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.
👍
Review code and test in runbot.
cc @Tecnativa