Skip to content

[10.0][FIX] Automatic sale workflow - Register payment multi-currency#597

Merged
pedrobaeza merged 1 commit intoOCA:10.0from
acsonefho:fix_multi_currency_automatic_payment
Apr 23, 2018
Merged

[10.0][FIX] Automatic sale workflow - Register payment multi-currency#597
pedrobaeza merged 1 commit intoOCA:10.0from
acsonefho:fix_multi_currency_automatic_payment

Conversation

@acsonefho
Copy link
Contributor

@acsonefho acsonefho commented Jan 25, 2018

How to reproduce

  • Create an invoice with another currency than your invoice company.
  • Use the automatic workflow (with auto-payment)
  • Check the invoice payment (it's not full paid, depending on the currency rate).

If you don't use the automatic workflow, you can open the payment wizard and this one ask you the currency to use (by default, Odoo put the currency of the company).

Patch

  • Use the currency of the invoice into the invoice payment wizard
  • Now payment amount match on the invoice

@acsonefho acsonefho force-pushed the fix_multi_currency_automatic_payment branch from be72825 to 5eee167 Compare January 26, 2018 08:31
@acsonefho acsonefho changed the title [10.0][FIX] Automatic sale workflow - Register payment [10.0][FIX] Automatic sale workflow - Register payment multi-currency Jan 26, 2018
@yajo yajo added this to the 10.0 milestone Feb 19, 2018
@fclementic2c
Copy link
Member

👍

Copy link
Contributor

@jcoux jcoux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Copy link
Contributor

@Tonow-c2c Tonow-c2c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Code review only

@pedrobaeza pedrobaeza merged commit b637592 into OCA:10.0 Apr 23, 2018
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.

6 participants