OCA / l10n-spain

Odoo Spain Localization
https://www.aeodoo.org/estado-localizacion
GNU Affero General Public License v3.0
281 stars 518 forks source link

[15.0][IMP+FIX] l10n_es_payment_order_confirming_aef: Cambios en archivo de confirming #3623

Closed victoralmau closed 3 months ago

victoralmau commented 4 months ago

Cambios hechos:

  1. 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:

  1. 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 3 months ago

This PR looks fantastic, let's merge it! Prepared branch 15.0-ocabot-merge-pr-3623-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 3 months ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-3623-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 3 months ago

/ocabot merge minor

OCA-git-bot commented 3 months ago

This PR looks fantastic, let's merge it! Prepared branch 15.0-ocabot-merge-pr-3623-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 3 months ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-3623-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 3 months ago

/ocabot merge minor

OCA-git-bot commented 3 months ago

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

OCA-git-bot commented 3 months ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-3623-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 3 months ago

/ocabot merge minor

OCA-git-bot commented 3 months ago

This PR looks fantastic, let's merge it! Prepared branch 15.0-ocabot-merge-pr-3623-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 3 months ago

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