[16.0][MIG] l10n_es_dua_sii: Migration to version 16.0 #3093
Conversation
Currently translated at 100.0% (2 of 2 strings) Translation: l10n-spain-10.0/l10n-spain-10.0-l10n_es_dua_sii Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-10-0/l10n-spain-10-0-l10n_es_dua_sii/gl/
* README update. Not related but rechecking the roadmap was not accurated * Tests adapted * Tax description renamed
…not been defined The "Fiscal position" field of the invoice is not a mandatory field so it might be not defined, in this case, its default value is "Régimen nacional". The original module assumes that the fiscal position of the DUA always exists and this may not be so, mainly in multicompany environments where not all companies use DUA and they don't need to have this fiscal position created.
Se quita ImporteTotal como informacion que se manda al SII, para evitar problemas de nuevas validaciones incorporadas en el SII para 2021
- Refactor and improve tests
- 10n_es_aeat_sii_oca: adapt helper method
- Allow for defining the base module from which to import the expected
output json to allow usage downstream.
- Return created invoice
TT26766
Co-Authored-By: Pedro M. Baeza <[email protected]>
Currently translated at 100.0% (2 of 2 strings) Translation: l10n-spain-14.0/l10n-spain-14.0-l10n_es_dua_sii Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-14-0/l10n-spain-14-0-l10n_es_dua_sii/ca/
Moved so that they are not listed at the beginning of the test folder, and thus it's easier to locate test files.
Co-authored-by: Fernando La Chica <[email protected]>
Mantener el tipo LC en el caso de que una factura sea DUA y a la vez se le marque la casilla de Liquidación complementaria. Cambios traídos de la v14.0 OCA#2358
|
ping @pedrobaeza |
|
/ocabot migration l10n_es_dua_sii |
|
¿Podríamos mejorar el README o los comentarios en el código de la función que decide lo que se presenta y lo que no? Estuvimos analizando @HaraldPanten y @cubells (ellos más bien) el código y les costó entender qué hacía la función En mi opinión lo que hace es: Mira la posición fiscal y si es Importación con DUA, luego mira si tienes impuestos mapeados al SII (como del de importación 21%) y si los tienes entonces la manda al SII pero sólo esas líneas con estos impuestos. |
There was a problem hiding this comment.
A ver, lo veo bien.
He encontrado al probar esto no related (creo): #3096
Y luego, pues que claro no se puede probar el módulo porque al mandar al SII
File "/opt/odoo-venv/lib/python3.10/site-packages/requests/adapters.py", line 458, in send
self.cert_verify(conn, request.url, verify, cert)
File "/opt/odoo-venv/lib/python3.10/site-packages/requests/adapters.py", line 285, in cert_verify
raise OSError(
OSError: Could not find the TLS certificate file, invalid path: /opt/certificates/publicCert.crt
Pero lo veo bien 👍🏼
|
Hola Rafa, Sí, el error es normal al no detectar el certificado. Para hacer la comprobación al 100% ya es mejor montarlo en local con un certificado y probar envíos de test. Es la mejor prueba que se podría hacer "sin causar daños", en mi opinión. En cuanto al README, ¿tenías algo pensado? Es decir... se puede mejorar con 4 explicaciones extra; pienso lo mismo. Si quieres, cuando esté fusionado, enviáis un [IMP] del README (en V14) y lo pasamos a V15 y V16. ¿Cómo lo ves? Saludos. |
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 84a7bfb. Thanks a lot for contributing to OCA. ❤️ |
@Tecnativa TT43232