-
-
Notifications
You must be signed in to change notification settings - Fork 529
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[12.0][IMP] Crea método para obtener si una factura se enviará como simplificada o completa #1409
Conversation
Hi @pedrobaeza, |
Me parece bien, pero haz squash de los commits en uno y no es necesario que cambies la versión del módulo. Ya lo hará el bot al fusionar y así se evitan posibles conflictos. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gracias por los tests extra
@jalzaga vas a poder revisar el PR? |
Debes hacer rebase y no pull |
Lo había corregido desde la interfaz de github, será por eso. |
Uf, por favor, huye de esa opción, porque estropea todo. Cógelo en local y haz squash. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM a la espera de squash
¿Puede estar el travis cascado? Aquí se ve sin completar, pero al entrar indica todo ok. |
Sí, alguna vez le pasa. De todas formas, el bot lanza otro CI: /ocabot merge patch |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at 89ae496. Thanks a lot for contributing to OCA. ❤️ |
Perfect, gracias. |
Al hilo de: #1171
La idea de este PR es facilitar la modificación de la forma en que se obtiene si una factura es simplificada o no.
Se veis el link anterior, hay casos en los que es mejor controlar a nivel de factura si cuando se envía al SII se hace como simplificada o no, y no a nivel de partner.
Este cambio permitiría heredar el método y cambiar la lógica para decidir si una factura es simplificada o no, permitiendo heredarlo y no tener que modificar sustancialmente los métodos en los que se comprueba el partner.sii_simplified_invoice
Entiendo que esto no afecta a la operativa actual, pero si mejora la integración con otras casuísticas.
A ver qué os parece.
Un saludo.