search

OCA / bank-payment

Odoo Electronic Payment
https://odoo-community.org/psc-teams/banking-10
GNU Affero General Public License v3.0
198 stars 524 forks source link

[17.0][IMP] account_payment_order*: Define the correct date in the files that are generated #1319

Closed victoralmau closed 1 month ago

victoralmau commented 2 months ago

FWP from 16.0: https://github.com/OCA/bank-payment/pull/1315

Define the correct date in the files that are generated.

Compatibility with https://github.com/OCA/bank-payment/pull/1306

The payment (account.payment) and the account entry (account.move) are defined with today's date, but the "important" date is the date of the line, therefore the payment_line_date field is created.

The modules account_banking_sepa_sepa_direct_debit + account_banking_sepa_credit_transfer to use the payment_line_date field.

The modules l10n_es_payment_order_confirming_aef + l10n_es_payment_order_confirming_sabadell of OCA/l10n-spain do not need these changes because they do not use payment_ids.

Please @pedrobaeza can you review it?

@Tecnativa TT49988

Raul-S73 commented 2 months ago

Hello @victoralmau, are you planning to do it for version 14.0?

victoralmau commented 2 months ago

Already done in 14.0: https://github.com/OCA/bank-payment/pull/1311

pedrobaeza commented 1 month ago

/ocabot merge patch

OCA-git-bot commented 1 month ago

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

OCA-git-bot commented 1 month ago

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