Skip to content
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

[FEAT] mrp_stock_analytic: add analytic to new raw lines #687

Open
wants to merge 13 commits into
base: 17.0
Choose a base branch
from

Conversation

aisopuro
Copy link
Contributor

@aisopuro aisopuro commented Sep 5, 2024

Depends on:

Built on top of the dependent PR, only the last commit is actually different.

aisopuro and others added 13 commits September 5, 2024 18:54
Needed to also rename method starting with double-underscrore to
avoid Python method name mangling.
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
Switched to using shared base class instead
mrp recently added additional things into the context in a way
that broke the test. It was over-engineered anyway, we can make
do with a simpler approach.
Also add analytic distribution when manually adding raw lines to an MO
with an analytic distribution set.
@aisopuro aisopuro force-pushed the mrp_stock_analytic_set_analytics_on_new_raw_lines branch from fb0c8e6 to 88e2b33 Compare September 5, 2024 16:04
aisopuro added a commit to avoinsystems/account-analytic that referenced this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants