Skip to content

Conversation

@victoralmau
Copy link
Member

FWP from 16.0: #1315

Define the correct date in the files that are generated.

Compatibility with #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

…at are generated.

Compatibility with OCA#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.

TT49988
@Raul-S73
Copy link

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

@victoralmau
Copy link
Member Author

Already done in 14.0: #1311

@pedrobaeza pedrobaeza added this to the 17.0 milestone Jul 16, 2024
@pedrobaeza
Copy link
Member

/ocabot merge patch

@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-1319-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit aa24325 into OCA:17.0 Aug 9, 2024
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 17.0-imp-account_payment_order-TT49988 branch August 9, 2024 18:33
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