Joan Sisquella
Joan Sisquella
Hi @pedrobaeza , The current approach of the account_tax_balance runs first this direct SQL for selecting ids of the used taxes: ``` SELECT id FROM account_tax at WHERE company_id =...
With this new approach, we have reduced the time for reporting a month from 5 minutes 30 secs to only 15 secs.
I have removed pandas and natsort dependencies. Performance has been a little bit affected, not critical at all, that addition wasn't essential. Thanks for your review!
Hi @elbati, This PR is not going to be merged in v12 because of all the dependent modules, so these dependent modules don't need to be modified in v12.
@pedrobaeza yes, I am forward-porting the fixed issues and improvements by making PR to v13. I try to backport other improvements done in v13 to this PR too. Thank you...
Hello @Levi-Bensley, This problem was fixed in v13 in this PR #682, that is already merged. Which is a forward port of this PR in v12 #660. Thanks for your...
@dreispt Is it possible to reopoen and merge? I think it is a needed fix for the module. Thank you
@dreispt Yes, of course. Done!
You are right @AaronHForgeFlow , I've just applied what you suggest
@LoisRForgeFlow comments attended