OCA / account-analytic

Odoo Account Analytic Related Addons
GNU Affero General Public License v3.0
101 stars 358 forks source link

[16.0][ADD] analytic_mixin_analytic_accounts_and_plans #673

Open norlinhenrik opened 1 month ago

norlinhenrik commented 1 month ago

This PR is based on #565. See also PR 1207 for account_financial_report.

account.move.line new fields:

The compute method in account_financial_report (AGPL) is written for stored fields, with good performance. The compute method in #565 (LGPL) is written for non-stored fields.

QUESTIONS 1) If account_financial_report depends on this module for analytic_account_ids, will the performance suffer? 2) If no dependency, and both modules introduce analytic_account_ids, can both be used in the same database? 3) What should be the module name?

pedrobaeza commented 1 month ago

Please avoid plurals in module names, and I would say that this requires a shorter name that summarizes the goal of the module.

norlinhenrik commented 1 month ago

I suggested "accounts" and "plans" since they are many2many fields. But I am fine with "account" and "plan".

What would be a good shorter name?

pedrobaeza commented 1 month ago

What about analytic_unfolding_mixin?