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
[IMP] services: avoid exporting labels of technical fields #162167
[IMP] services: avoid exporting labels of technical fields #162167
Conversation
dda2cbb
to
5f0b068
Compare
5f0b068
to
3370678
Compare
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 again for your work, I left some comments. 🙂
@@ -54,19 +54,19 @@ def _domain_employee_id(self): | |||
'project.task', 'Task', index='btree_not_null', | |||
compute='_compute_task_id', store=True, readonly=False, | |||
domain="[('allow_timesheets', '=', True), ('project_id', '=?', project_id)]") | |||
parent_task_id = fields.Many2one('project.task', related='task_id.parent_id', store=True) | |||
parent_task_id = fields.Many2one('project.task', related='task_id.parent_id', store=True, export_string_translation=False) |
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 field seems used in the search view, so I guess we should still keep the string translated in that case, right? 🧐
@@ -10,7 +10,7 @@ class Project(models.Model): | |||
|
|||
allow_timesheets = fields.Boolean( | |||
"Timesheets", compute='_compute_allow_timesheets', store=True, readonly=False, | |||
default=True) | |||
default=True, export_string_translation=False) |
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 field seems display in the UI and so I don't think it is a good idea to not translate its string.
What do you think? 🧐
3370678
to
a8e3ef7
Compare
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.
LGTM, thanks for your work. 🙂
robodoo r+
@MissingNoShiny @xavierbol unable to stage: merge conflict |
a8e3ef7
to
9ff900b
Compare
robodoo r+ |
9ff900b
to
1698997
Compare
1698997
to
bdf813e
Compare
robodoo r+ |
Task-3631801 closes #162167 Related: odoo/enterprise#60904 Signed-off-by: Xavier Bol (xbo) <xbo@odoo.com>
Task-3631801