-
-
Notifications
You must be signed in to change notification settings - Fork 429
[18.0][MIG] mrp_stock_analytic #854
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
base: 18.0
Are you sure you want to change the base?
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
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 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. |
Supersede: #793
@ForgeFlow