search

OCA / l10n-argentina

Odoo modules for Argentina
GNU Affero General Public License v3.0
22 stars 51 forks source link

[14.0] [IMP] l10n_ar_ledger: VAT reports #79

Closed ibuioli closed 1 year ago

OCA-git-bot commented 1 year ago

Hi @nimarosa, some modules you are maintaining are being modified, check this out!

nimarosa commented 1 year ago

@ibuioli Igna, que bueno que pudiste subir los reportes.

El error de pre commit refiere a que seguro tenes configurado VsCode o lo que estén usando para hacer el xml, para usar tabulaciones en vez de 4 espacios que es el estándar de odoo.

Si cambias esa configuración y forzas reformat en los archivos debería arreglarlo.

ibuioli commented 1 year ago

@nimarosa Si, lo pude formatear bien, ahora veo que hay algo de tabs pero en dos lineas; también me está dando un error en un archivo de CRON del l10n_ar_afipws_fe ese no lo toqué. Ahora voy a corregir lo del Ledger.

ibuioli commented 1 year ago

@nimarosa ahi fue bien

ibuioli commented 1 year ago

@nimarosa esto lo que falló fue los test que no están programados, se puede hacer el merge igual? a mi no me deja

nimarosa commented 1 year ago

/ocabot merge patch

OCA-git-bot commented 1 year ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 14.0-ocabot-merge-pr-79-by-nimarosa-bump-patch, awaiting test results.

OCA-git-bot commented 1 year ago

Congratulations, your PR was merged at 4269fa89988b74f82cc33a906b457268ad43149f. Thanks a lot for contributing to OCA. ❤️