Cambios Varios

[AlmustafaNET]

• Se guarda el certificado y el private_key en la BDD, esto hace que la firma sea 4 veces más rápida.
• Aumento de campos en el EDI Document, para facilitar el formulario de anulacion del SRI.
• Ambiente=none , sirve para usar el sistema sin enviar nada al SRI.

[celm1990]

@AlmustafaNET

Gracias por el PR, te dejo mis comentarios

• Mantengamos los commits separados y no mezclarlos con otros cambios, veo que aca estas mezclando lo de Retenciones que esta en otro PR
• Verifica que el pre-commit y los test se ejecuten correctamente, eso nos garantiza que los cambios no afecten lo que ya existe
• Sigamos las Guideslines al momento de hacer los commits, hagamos squash para dejar los commits necesarios, en este documento estan los enlaces a la mayoria de documentacion relacionada https://docs.google.com/document/d/1GFfbsBJEFaOn4tF_D2LFvvi2QTUT1BmYHsrn6YascbY/edit?usp=sharing

[AlmustafaNET]

Buen día Es la primera vez que trabajo con estos requerimientos, disculparán nomás, ya voy aprendiendo. Lo tengo en el repositorio: https://github.com/AlmustafaNET/l10n-ecuador Adjunto algunas imágenes de referencia para que vean como lo tengo. - Lineas de la retención [image: image.png] - Movimientos contables [image: image.png] En los pagos de la factura, nos indica q ese pago es de una retención (RETV): [image: image.png] Estos son los movimientos de la retención en compras: [image: image.png] Esta probado con el EDI, PDF, y envio al SRI. Me salen errores en los TEST, mañana los reviso para poder hacer un PULL request. Saludos El lun, 25 dic 2023 a las 12:07, Carlos Lopez ***@***.***>) escribió:

…

@AlmustafaNET <https://github.com/AlmustafaNET> Gracias por el PR, te dejo mis comentarios - Mantengamos los commits separados y no mezclarlos con otros cambios, veo que aca estas mezclando lo de Retenciones que esta en otro PR - Verifica que el pre-commit y los test se ejecuten correctamente, eso nos garantiza que los cambios no afecten lo que ya existe - Sigamos las Guideslines al momento de hacer los commits, hagamos squash para dejar los commits necesarios, en este documento estan los enlaces a la mayoria de documentacion relacionada https://docs.google.com/document/d/1GFfbsBJEFaOn4tF_D2LFvvi2QTUT1BmYHsrn6YascbY/edit?usp=sharing — Reply to this email directly, view it on GitHub <#59 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACRGHFU2VCBXTWHL4XN6PA3YLGXEDAVCNFSM6AAAAABA6U5RWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRZGA2TOMRVHA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[AlmustafaNET]

A espera de algún comentario de su parte, sigo con las actualizaciones. Primero, ya hice el pull-request a: Odoo-EC/l10n-ecuador/17.0-l10n_ec_withhold que es de donde hice el fork donde estoy trabajando, con todos los cambios de las retenciones que les comente en el correo anterior. Segundo, agregue en el l10n_ec_account_edi un mensaje de alerta para cuando la firma electrónica esta por caducar, sale así: [image: image.png] Me gustaría saber si lo que estoy haciendo sirve para el desarrollo en conjunto o si lo hago por mi cuenta . De ser la primera opción, cuando se reúnen para conversar sobre los próximos cambios o en que nomás se va a trabajar. El lun, 8 ene 2024 a las 18:38, Almustafa ***@***.***>) escribió:

…

Buen día Es la primera vez que trabajo con estos requerimientos, disculparán nomás, ya voy aprendiendo. Lo tengo en el repositorio: https://github.com/AlmustafaNET/l10n-ecuador Adjunto algunas imágenes de referencia para que vean como lo tengo. - Lineas de la retención [image: image.png] - Movimientos contables [image: image.png] En los pagos de la factura, nos indica q ese pago es de una retención (RETV): [image: image.png] Estos son los movimientos de la retención en compras: [image: image.png] Esta probado con el EDI, PDF, y envio al SRI. Me salen errores en los TEST, mañana los reviso para poder hacer un PULL request. Saludos El lun, 25 dic 2023 a las 12:07, Carlos Lopez ***@***.***>) escribió: > @AlmustafaNET <https://github.com/AlmustafaNET> > > Gracias por el PR, te dejo mis comentarios > > - Mantengamos los commits separados y no mezclarlos con otros > cambios, veo que aca estas mezclando lo de Retenciones que esta en otro PR > - Verifica que el pre-commit y los test se ejecuten correctamente, > eso nos garantiza que los cambios no afecten lo que ya existe > - Sigamos las Guideslines al momento de hacer los commits, hagamos > squash para dejar los commits necesarios, en este documento estan los > enlaces a la mayoria de documentacion relacionada > https://docs.google.com/document/d/1GFfbsBJEFaOn4tF_D2LFvvi2QTUT1BmYHsrn6YascbY/edit?usp=sharing > > — > Reply to this email directly, view it on GitHub > <#59 (comment)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/ACRGHFU2VCBXTWHL4XN6PA3YLGXEDAVCNFSM6AAAAABA6U5RWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRZGA2TOMRVHA> > . > You are receiving this because you were mentioned.Message ID: > ***@***.***> >

[celm1990]

Hola, las reuniones son Lunes y miercoles via Jitsi, dejo el link x aca
https://meet.jit.si/Migraci%C3%B3nOdoo15

Respecto a los cambios, la sugerencia es que los cambios se hagan por separado cada caso, lo que corresponda a Retenciones en una rama puntual, lo que corresponda a firma electronica en otro PR y asi un PR por cada funcionalidad independiente, ya que lo de retenciones aun esta bajo desarrollo y podria tomar mas tiempo fusionarla con las ramas principales, en cambio lo de firma se podria revisar mucho mas rapido por separado

Los comentarios/revisiones se harian en cada PR

[AlmustafaNET]

Muy buenas. - Lo de la fecha es para hacer una comparación y sacar días, por lo tanto la zona horaria no afectaría en lo más mínimo. Pero comprendo la necesidad de estandarizar todo, así que ya lo cambio. - Como le hago ahora para separar los comits ?, o que me recomiendas hacer?, para actualizar los repositorios l10n_ec_account_edi, hago el pull request a Odoo-EC/l10n-ecuador/17.0 ?, o a la del OCA ? Saludos, y hoy en la noche me conecto. El mar, 9 ene 2024 a las 18:48, Carlos Lopez ***@***.***>) escribió:

…

Hola, las reuniones son Lunes y miercoles via Jitsi, dejo el link x aca https://meet.jit.si/Migraci%C3%B3nOdoo15 Respecto a los cambios, la sugerencia es que los cambios se hagan por separado cada caso, lo que corresponda a Retenciones en una rama puntual, lo que corresponda a firma electronica en otro PR y asi un PR por cada funcionalidad independiente, ya que lo de retenciones aun esta bajo desarrollo y podria tomar mas tiempo fusionarla con las ramas principales, en cambio lo de firma se podria revisar mucho mas rapido por separado Los comentarios/revisiones se harian en cada PR — Reply to this email directly, view it on GitHub <#59 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACRGHFXH7QCWNUZZXD4VJNTYNXJLHAVCNFSM6AAAAABA6U5RWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBTHE3TCMBWGM> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[celm1990]

Hola, mis comentarios entre lineas:

Muy buenas.

• Lo de la fecha es para hacer una comparación y sacar días, por lo tanto la zona horaria no afectaría en lo más mínimo. Pero comprendo la necesidad de estandarizar todo, así que ya lo cambio.
• Como le hago ahora para separar los comits ?, o que me recomiendas hacer?,

Es un tema de git, puedes crear una nueva rama y hacer cherry-pick del commit que quieras pasar de una rama a otra y asi con los demas commits, ahora que si dentro de 1 mismo commit se han mezclado muchos cambios entonces ahi se complica algo mas y tendrias que hacer algun rebase y un reset --soft para ir separando cosas en commits individuales, pero con todo es un tema de git, revisa la documentacion o lo revisamos en alguna session puntual

• para actualizar los repositorios l10n_ec_account_edi, hago el pull request a Odoo-EC/l10n-ecuador/17.0 ?, o a la del OCA ?

Si los cambios son a modulos que ya estan en la rama 17.0 los PR que vengan directo aca a la OCA. Solo lo que tiene que ver con retenciones tratemoslo en Odoo-EC xq esto no esta aun fusionado en la OCA.