Skip to content

Conversation

AungKoKoLin1997
Copy link
Contributor

@AungKoKoLin1997 AungKoKoLin1997 commented Sep 3, 2025

This PR adds an option to display the exchange rate either as company currency per unit or unit per company currency.

Example (company currency: JPY; 1 USD = 150 JPY):

  • (Default) Unit per Company Currency → 0.0066 (USD/JPY)
  • Company Currency per Unit → 150 (JPY/USD)

@qrtl QT5684

@OCA-git-bot
Copy link
Contributor

Hi @victoralmau,
some modules you are maintaining are being modified, check this out!

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-imp-account_invoice_show_currency_rate branch 2 times, most recently from cbe79da to 7383b2f Compare September 3, 2025 04:37
@AungKoKoLin1997 AungKoKoLin1997 marked this pull request as ready for review September 3, 2025 04:37
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-imp-account_invoice_show_currency_rate branch from 7383b2f to 7c73788 Compare September 3, 2025 07:25
@AungKoKoLin1997 AungKoKoLin1997 changed the title [16.0][IMP] account_invoice_show_currency_rate: add option to show foreign currency per company currency [16.0][IMP] account_invoice_show_currency_rate: add option to choose currency rate display format Sep 3, 2025
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-imp-account_invoice_show_currency_rate branch 2 times, most recently from 68a7907 to c7014af Compare September 3, 2025 07:44
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-imp-account_invoice_show_currency_rate branch 5 times, most recently from bda8dd6 to 22d485b Compare September 4, 2025 04:05
…cy rate display format

This commit adds an option to display the exchange rate either as company currency per unit or unit per company currency.

Example (company currency: JPY; 1 USD = 150 JPY):

- (Default) Unit per Company Currency → 0.0066 (USD/JPY)
- Company Currency per Unit → 150 (JPY/USD)
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-imp-account_invoice_show_currency_rate branch from 22d485b to d01904d Compare September 5, 2025 07:13
Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

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

LGTM. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants