documentation icon indicating copy to clipboard operation
documentation copied to clipboard

[IMP]l10n_es: Tax report: Modelo 130

Open igbe-odoo opened this issue 1 year ago • 7 comments

Add the modelo 130 report and the corresponding BOE files export for spain localization.

Add also some documentation for using the report.

task-3866238

igbe-odoo avatar Jun 12 '24 15:06 igbe-odoo

Pull request status dashboard

robodoo avatar Jun 12 '24 15:06 robodoo

Hi @igbe-odoo! Thanks for your great work. I pushed some changes to comply with documentation guidelines and removed some screenshots that were not necessary.

@odoo/be-doc-review From what I can tell, the R&D task has been not merged yet. Might be best to give r+ rights.

toaa-odoo avatar Jun 20 '24 11:06 toaa-odoo

Hey @igbe-odoo, I noticed that the (formatting) changes I pushed while reviewing this PR were overwritten and reverted. Just wanted to check in and see if there was a specific reason for that? Thanks!

auva-odoo avatar Jun 28 '24 09:06 auva-odoo

i presume he forgot to pull the branch before force pushing on it :grimacing:

malb-odoo avatar Jun 28 '24 09:06 malb-odoo

Indeed I made a little mistake while rebasing my branche I'm sorry about that I didn't see it, I will cherry pick your commit and reset the branch in the state she was before. Thanks you for noticing me the issue and please excuse my little mistake, mb on this :/

igbe-odoo avatar Jun 28 '24 10:06 igbe-odoo

No worries at all, it happens! :slightly_smiling_face:

auva-odoo avatar Jun 28 '24 11:06 auva-odoo

It should be good now 😉

igbe-odoo avatar Jun 28 '24 11:06 igbe-odoo

Don't know if the delegate still works with the different force push, can we have a new delegate if it's not the case :smile:

malb-odoo avatar Jul 19 '24 08:07 malb-odoo

@igbe-odoo You have the delegated rights, so you have to r+ this one

william-andre avatar Aug 02 '24 14:08 william-andre

@robodoo r+

igbe-odoo avatar Aug 02 '24 15:08 igbe-odoo