Henrik Norlin

Results 55 comments of Henrik Norlin

My bank statement has negative numbers in the second column. Do you think we should rename the field to `amount_negative_column`?

Waiting to merge `analytic_mixin_analytic_account`, see #565.

Maybe a better implementation would be to create an `analytic_category_distribution` JSON field on the `account.move.line`? EDIT Two observations in 16.0: - `account.analytic.plan` has `parent_id`. Having one parent for all plans...

> I've chosen to keep non-stored fields in this module for two main reasons: > > * **Technical**: Due to the reasons previously mentioned. (We might, however, force stored fields...

After merging, please review #630 which depends on this PR.

@sbidoul Would you like to review?

What do you think about computing 2 more fields: `analytic_plan_ids` & `analytic_plan_names` ? If we can merge this PR, I can open a new PR with this improvement.

> On the other hand, It would be great to have it integrated at some point with others modules like this one - https://github.com/OCA/account-financial-reporting/blob/16.0/account_financial_report/models/account_move_line.py - to avoid having several logic...

@malekben @etobella Are you able to reproduce the error? I am not. I made a new bank journal. I made 2 invoices of the same product, to 2 different partners....

Hi @alexis-via, Maybe I should have opened an issue to discuss improvements, but I find it convenient to show my suggestions in code. Below is a short explanation of the...