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

[15.0][IMP] account_payment_order: Add Default Tranfer Move Date configuration #1289

Merged

Conversation

victoralmau
Copy link
Member

Add Default Tranfer Move Date configuration

Options:

  • Payment date: Set the payment date in the transfer entry (similar to the existing so far).
  • Now + Payment due date: Set now date the payment date in the transfer entry and t he corresponding due date in journal items.

ejemplo

Please @pedrobaeza @carlosdauden and @sergio-teruel can you review it?

@Tecnativa TT49582

@pedrobaeza pedrobaeza added this to the 15.0 milestone Jun 14, 2024
account_payment_order/models/account_payment_mode.py Outdated Show resolved Hide resolved
account_payment_order/models/account_payment_mode.py Outdated Show resolved Hide resolved
account_payment_order/models/account_payment_mode.py Outdated Show resolved Hide resolved
account_payment_order/models/account_payment.py Outdated Show resolved Hide resolved
account_payment_order/models/account_payment.py Outdated Show resolved Hide resolved
Options:
- Payment date: Set the payment date in the transfer entry (similar to the existing so far).
- Now + Payment date as due date: Set now date the payment date in the transfer entry and t
he corresponding due date in journal items.

TT49582
@victoralmau victoralmau force-pushed the 15.0-imp-account_payment_order-TT49582 branch from c2f612b to 5e283b8 Compare June 17, 2024 06:23
@victoralmau
Copy link
Member Author

Changes done.

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 minor

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot merged commit 6d8f1cd into OCA:15.0 Jun 17, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 15.0-imp-account_payment_order-TT49582 branch June 17, 2024 14:11
@pedrobaeza
Copy link
Member

Please fw-port it to v16

victoralmau added a commit to Tecnativa/bank-payment that referenced this pull request Jun 17, 2024
…on (now_date_maturity)

Related to OCA#1289

In v13 the existing behavior was defined with today's date the Transfer Move, therefore, we define
that behavior by default without configuration since it is the expected behavior.

https://github.com/OCA/bank-payment/pull/805/files

TT49582
victoralmau added a commit to Tecnativa/bank-payment that referenced this pull request Jun 17, 2024
…on (now_date_maturity)

Related to OCA#1289

In v13 the existing behavior was defined with today's date the Transfer Move, therefore, we define
that behavior by default without configuration since it is the expected behavior.

https://github.com/OCA/bank-payment/pull/805/files

TT49582
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