[16.0][ADD] account_analytic_category (replace account.analytic.group) #630
Conversation
norlinhenrik
force-pushed
the
16.0-add-account_analytic_category
branch
6 times, most recently
from
36d2609
to
56df5b1
Compare
| This module introduces account.analytic.category | ||
| as a replacement for account.analytic.group which was removed in Odoo 16.0. |
There was a problem hiding this comment.
Is there a reason to rename the model?
There was a problem hiding this comment.
Odoo called the model for Analytic Categories and I thought maybe this is a better name. But I am flexible on this.
| "category": "Account", | ||
| "website": "https://github.com/OCA/account-analytic", | ||
| "depends": [ | ||
| "account_financial_report", # account.move.line.analytic_account_ids |
There was a problem hiding this comment.
I thought this dependency was a bit heavy. Maybe the part that depends on account.move.line.analytic_account_ids can be handled in a separate glue module in the account-financial-reporting repo.
There was a problem hiding this comment.
I made a new module account_move_line_analytic_account_ids.
There was a problem hiding this comment.
There is also this PR #565 but the proposed solution in this PR doesn't store analytic_account_ids due to a technical reason.
There was a problem hiding this comment.
Nice :) I have added analytic_mixin_analytic_account as a dependeny.
norlinhenrik
force-pushed
the
16.0-add-account_analytic_category
branch
8 times, most recently
from
889f799
to
6c9651a
Compare
norlinhenrik
force-pushed
the
16.0-add-account_analytic_category
branch
2 times, most recently
from
1736718
to
04ca2fb
Compare
norlinhenrik
force-pushed
the
16.0-add-account_analytic_category
branch
from
04ca2fb
to
bec164d
Compare
|
Waiting to merge |
|
Maybe a better implementation would be to create an EDIT
|
2 new modules with 2 new fields on
account.move.lineto use in pivot:account_move_line_analytic_account_ids: new fieldanalytic_account_ids.account_analytic_category: new fieldanalytic_category_ids.