documentation icon indicating copy to clipboard operation
documentation copied to clipboard
verified publisher icon odoo

[ADD] Payroll: Salary attachment report

Open larm-odoo opened this issue 1 year ago • 2 comments

Adding the salary attachment reporting doc as a separate stand-alone doc instead of combined in a reporting doc with all reports on it.

Original task card for this PR.

larm-odoo avatar Sep 19 '24 21:09 larm-odoo

Pull request status dashboard

robodoo avatar Sep 19 '24 21:09 robodoo

Hi @dikd-odoo - this is ready for a first-round review!

larm-odoo avatar Oct 24 '24 19:10 larm-odoo

Hi @dikd-odoo - thank you for your review! I put in a comment from one of your edits, that the report is a default one. If you go to a 17.0 runbot, there should be info in there, and the report is visible. LMK if you have any issues finding it!

larm-odoo avatar Oct 28 '24 21:10 larm-odoo

Hi @larm-odoo thank you for the clarification! I checked the flow in a 17.0 runbot and it went smoothly. This should be ready to go, with the note that the 18.0 will be updated to reflect configuration changes (Salary Attachment Report not a pre-configured report).

Let me know if you have any questions, thank you!

dinakdugar avatar Oct 28 '24 22:10 dinakdugar

Hi @samueljlieber - this is ready for a tech review when you have a chance.

larm-odoo avatar Oct 29 '24 18:10 larm-odoo

@robodoo r+

larm-odoo avatar Oct 31 '24 17:10 larm-odoo