[16.0][IMP+FIX] l10n_es_payment_order_confirming_aef: Cambios en archivo de confirming

victoralmau commented 5 months ago

FWP from 15.0: https://github.com/OCA/l10n-spain/pull/3623

Cambios hechos:

[x] [IMP] Añadir campo Modalidad remesa
[x] [FIX] Cuenta bancaria en posición correcta

Añadir campo Modalidad remesa

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.

Links documentación:

Cuenta bancaria en posició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".

@Tecnativa TT49479 TT49550

OCA-git-bot commented 5 months ago

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

OCA-git-bot commented 5 months ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-3642-by-pedrobaeza-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

pedrobaeza commented 5 months ago

/ocabot merge minor

OCA-git-bot commented 5 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-3642-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 5 months ago

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

OCA / l10n-spain

[16.0][IMP+FIX] l10n_es_payment_order_confirming_aef: Cambios en archivo de confirming #3642