Skip to content

Conversation

@BernatObrador
Copy link
Contributor

@BernatObrador BernatObrador commented Oct 6, 2025

Esta PR agrega las referencias para emisor y receptor.

También agrega un campo para indicar la cuenta de factoring, que será usada como la cuenta bancaria en lugar de la del cliente, cuando el modo de pago sea de tipo especial.

@etobella @pedrobaeza, estos con los cambios que he realizado en referencia a esta issue.

Ya me comentáis que os parece.

Muchas gracias por vuestro tiempo.

@OCA-git-bot
Copy link
Contributor

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

Copy link
Member

@etobella etobella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BernatObrador Puedes añadir esta funcionalidad en un módulo dependiente y no directamente en el facturae? Estamos añadiendo una funcionalidad que generalmente no es necesaria. Ha ejemplos, como l10n_es_facturae_sale

@BernatObrador BernatObrador force-pushed the 17.0-imp-l10n_es_facturae-factoring branch from e10d65a to 3cc4b65 Compare October 6, 2025 08:06
@BernatObrador BernatObrador changed the title [IMP] l10n_es_facturae: Added fields for special(13) payment mean [17.0][ADD] l10n_es_facturae_special_payment: New module l10n_es_facturae_special_payment Oct 6, 2025
@BernatObrador BernatObrador force-pushed the 17.0-imp-l10n_es_facturae-factoring branch from 3cc4b65 to 29bc821 Compare October 6, 2025 08:16
@BernatObrador
Copy link
Contributor Author

@etobella Hecho!

¿Puedes revisarlo ahora?

Gracias.

@BernatObrador BernatObrador force-pushed the 17.0-imp-l10n_es_facturae-factoring branch from 29bc821 to a32bd2a Compare October 7, 2025 05:39
Copy link
Member

@etobella etobella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A nivel técnico, todo OK.

@pedrobaeza pedrobaeza added this to the 17.0 milestone Oct 7, 2025
@BernatObrador BernatObrador force-pushed the 17.0-imp-l10n_es_facturae-factoring branch from a32bd2a to 800ab7a Compare October 7, 2025 07:45
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-4417-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a1a5ce5 into OCA:17.0 Oct 16, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants