documentation icon indicating copy to clipboard operation
documentation copied to clipboard

[IMP] accounting: full documentation of the new report engine

Open oco-odoo opened this issue 3 years ago • 4 comments

oco-odoo avatar Oct 03 '22 16:10 oco-odoo

@jcs-odoo I still need to go over the helpers in the code, but let me know what you think of this one. I tried not to go too technical and keep explaining what was important for someone creating his own report. Hopefully I was clear enough ! ^^

oco-odoo avatar Oct 03 '22 16:10 oco-odoo

Awesome @oco-odoo :) It requires a lot of work before we can merge it, though, but it's so great to have such content from you! It's a gold mine :) Thanks for participating in the user doc. That means a lot!

jcs-odoo avatar Oct 04 '22 10:10 jcs-odoo

Accounting doc review was pinged by the bot and the PR is not in draft, but is it finished?

@jcs-odoo's comment seems to suggest otherwise. Let me know :)

toaa-odoo avatar Oct 17 '22 15:10 toaa-odoo

@LoredanaLrpz Don't re-review yet, I still have a few comments to address later :)

toaa-odoo avatar Nov 04 '22 14:11 toaa-odoo

@LoredanaLrpz Don't re-review yet, I still have a few comments to address later :)

Done! You can review :)

toaa-odoo avatar Nov 08 '22 15:11 toaa-odoo

@LoredanaLrpz Final review for you ? :)

toaa-odoo avatar Nov 09 '22 13:11 toaa-odoo

@toaa-odoo Hey, I'm currently working on it, but I think you've already received the answers I wrote to my comments. I'll ping you as soon as I'm done :)

LoredanaLrpz avatar Nov 09 '22 14:11 LoredanaLrpz

@oco-odoo You might want to have a look at @william-andre's comments

toaa-odoo avatar Dec 06 '22 07:12 toaa-odoo

@oco-odoo @william-andre @tsb-odoo @jcs-odoo Should you have any additional comments, let me know now. Otherwise, I'm considering this PR ready for merge @AntoineVDV.

Forward to master.

toaa-odoo avatar Dec 19 '22 11:12 toaa-odoo

@robodoo r+

jcs-odoo avatar Dec 21 '22 15:12 jcs-odoo