Conversation
Currently translated at 100.0% (4 of 4 strings) Translation: account-analytic-16.0/account-analytic-16.0-mrp_stock_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-16-0/account-analytic-16-0-mrp_stock_analytic/it/
* analytic_distribution is constrained to be valid on the MO, so we don't need to validate on confirmation * The `states`-attribute on ORM-fields is deprecated, and should be moved to views * mrp_account adds the distribution to the views we want, so we can just add attributes instead of re-adding the field * mrp.production already has mixin.analytic, no need to re-add it * analytic_distribution is already in the MO list view, no need to change its default visibility
Switched to using shared base class instead
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: account-analytic-17.0/account-analytic-17.0-mrp_stock_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-17-0/account-analytic-17-0-mrp_stock_analytic/
Currently translated at 100.0% (2 of 2 strings) Translation: account-analytic-17.0/account-analytic-17.0-mrp_stock_analytic Translate-URL: https://translation.odoo-community.org/projects/account-analytic-17-0/account-analytic-17-0-mrp_stock_analytic/it/
|
Hi @anajuaristi, @luisDIXMIT what do you think about this PR? |
luisDIXMIT
left a comment
luisDIXMIT
left a comment
There was a problem hiding this comment.
Tested on runboat and code review, LGTM!
|
@AaronHForgeFlow What do you think about it? |
|
Hi @ThiagoMForgeFlow In production order there is a field analytic distribution many2many. I tried several ways (with a single distribution, adding several of them, consuming everything, making partials... ) I could not obtain any single analytic line created. Maybe I'm missing something but It seems it doesn't work for me. On the other side, there is another module project_mrp, directly included on Odoo V18 core that allows to set a project_id on the manufacturing order. On this case, if you set the project, system will create the cost analytic lines against the analytic account of the project so I don't know if both can be compatible or not the way it is. Please let me know. |
|
@ThiagoMForgeFlow I added new improvement on top of your PR. |
11 participants
Supersede: #793
@ForgeFlow