Skip to content
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

[17.0][MIG] l10n_es_payment_order_confirming_aef: Migration to 17.0 #3749

Merged

Conversation

victoralmau
Copy link
Member

Superseed #3653

Migration to 17.0

@Tecnativa TT50082

[UPD] Update l10n_es_payment_order_confirming_aef.pot

[UPD] README.rst
[UPD] Update l10n_es_payment_order_confirming_aef.pot

[UPD] README.rst

[UPD] README.rst
Translated using Weblate (Spanish)

Currently translated at 100.0% (17 of 17 strings)

Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_payment_order_confirming_aef
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_payment_order_confirming_aef/es/
…number

The field `ref` in the account.move.line doesn't always contain
the same value as the account.move, and what we want is the
invoice number, so let's use the move one in any case.

Co-Authored-By: Pedro M. Baeza <[email protected]>
… es obligatoria pero Bankinter si la pide

[BOT] post-merge updates
Bankinter is actually using the processing date indicated in the file for their
planning and using the first line payment date causes that the files generated
don't work with that bank.

As harmless for the rest of the banks, let's use the date where the file is generated
for this data.

Closes OCA#3425

[BOT] post-merge updates
Force the invoice to use the partner bank account indicated for the test
for having a correct result no matter the upstream code.

[BOT] post-merge updates

[BOT] post-merge updates
…mesa

El dato relativo a la modalidad de la reserva se estaba definiendo vacío,
es posible definirlo ahora según las diferentes opciones existentes según
la documentación del archivo de confirming.

TT49479
…ón correcta

Se debe especificar la cuenta bancaria en la posición correcta (103) por lo que
se debe utilizar ljust() para rellenar espacios al final de la cuenta bancaria.

En la documentación se identifica: "Los campos alfanuméricos se alinearán a la
izquierda (“Nombre“) completándose con espacios por la derecha".

TT49550

[UPD] Update l10n_es_payment_order_confirming_aef.pot

[BOT] post-merge updates

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_payment_order_confirming_aef
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_payment_order_confirming_aef/
…ing payment method in bank journal

[UPD] Update l10n_es_payment_order_confirming_aef.pot

[BOT] post-merge updates

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: l10n-spain-16.0/l10n-spain-16.0-l10n_es_payment_order_confirming_aef
Translate-URL: https://translation.odoo-community.org/projects/l10n-spain-16-0/l10n-spain-16-0-l10n_es_payment_order_confirming_aef/
@pedrobaeza
Copy link
Member

/ocabot migration l10n_es_payment_order_confirming_aef

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Oct 3, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 3, 2024
47 tasks
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

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-3749-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 7ba137d into OCA:17.0 Oct 3, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 17.0-mig-l10n_es_payment_order_confirming_aef branch October 3, 2024 07:29
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.

9 participants