[16.0][MIG] analytic brand

[pcastelovigo]

using oca-port

[marielejeune]

Please see change commit that was applied in v18.
I think this is relevant in v16 too, as Odoo changed analytic accounts to analytic distributions.
4b05ed0

[pcastelovigo]

Please see change commit that was applied in v18. I think this is relevant in v16 too, as Odoo changed analytic accounts to analytic distributions. 4b05ed0

yes, it seems it need to be adapted to distributions,
i'll try to do this week, thanks

[pcastelovigo]

@marielejeune i think its ready

[marielejeune]

Please follow migration guidelines: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-16.0

All the commit history is squashed, and the migration commit doesn't contain anything.

[pcastelovigo]

Please follow migration guidelines: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-16.0

All the commit history is squashed, and the migration commit doesn't contain anything.

misunderstood the guidelines, thank you for your patience

[marielejeune]

LGTM.

Note that I'm currently facing some issues/difficulties using this module in v18.
The reason is that the analytic distribution is defined on the brand, while other composants use the analytic distribution models.

I think I will modify the module to add the brand dimension inside analytic distribution models too. I think this is something that could be useful for v16 too.

[pcastelovigo]

LGTM.

Note that I'm currently facing some issues/difficulties using this module in v18. The reason is that the analytic distribution is defined on the brand, while other composants use the analytic distribution models.

I think I will modify the module to add the brand dimension inside analytic distribution models too. I think this is something that could be useful for v16 too.

I expected it to add automatically the analytic tag to account.moves using that brand but it does not work that way, it was also in the precedent versions.
I had code to add them with used with account_brand.. I'll check your work in the future

[marielejeune]

Hi @pcastelovigo,
Please check at my refactoring to use account analytic distribution models, ready here: #286