-
-
Notifications
You must be signed in to change notification settings - Fork 837
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_job_position migration #328
9.0 [MIG] partner_contact_job_position migration #328
Conversation
- Renamed crm_job_position to partner_contact_job_position. - Drop all CRM dependencies. - Improve README. - Add form view. - Rename one model. - Rename some fields. - Reorder module. - Remove pot. - Update translation. - Use new-style short license and copyright headers.
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.
Choose another icon different, because this one is used in partner_sector.
Squash "OCA transbot" and the "en.po" commits together.
Images | ||
------ | ||
|
||
* `Antiun Ingeniería S.L. <http://www.antiun.com>`_: Icon. |
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.
This icon link is not correct.
"version": "9.0.1.0.0", | ||
'category': 'Customer Relationship Management', | ||
"website": "http://www.antiun.com", | ||
'author': 'Serv. Tecnolog. Avanzados - Pedro M. Baeza, ' |
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.
Switch to Tecnativa
both website and author.
_description = "Job position" | ||
|
||
name = fields.Char(required=True, translate=True) | ||
parent_id = fields.Many2one( |
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.
Check recursion
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.
And add a test for that
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.
Well, if you use something like https://github.com/odoo/odoo/blob/af865c2e9ea1dc1890ef33add9a6e4bc657068e0/addons/hr/hr.py#L264, there's no need for the test, as this method is already tested.
cc @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.
Tested in runbot
@cubells, please make the changes I requested. |
5453285
to
fe2c96b
Compare
* Changed icon * Change author and web * Check recursion
f57815d
to
03bab9c
Compare
No description provided.