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

[FW][FIX] base: use time zone aware display field for currency rates #101640

Conversation

fw-bot
Copy link
Contributor

@fw-bot fw-bot commented Sep 29, 2022

Steps to reproduce:

  1. change system and admin's time zone to one with a different date than UTC
  2. enable multi-currency in the accounting module
  3. add rates for the current date in local time zone and UTC
  4. go to settings > currencies
  5. the rate for the UTC date is displayed

To fix this, we should use a time zone aware field in res_currency.

opw-2945108

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

Forward-Port-Of: #100123

Steps to reproduce:

1. change system and admin's time zone to one with a different
date than UTC
2. enable multi-currency in the accounting module
3. add rates for the current date in local time zone and UTC
4. go to settings > currencies
5. the rate for the UTC date is displayed

To fix this, we should use a time zone aware field in `res_currency`.

opw-2945108

X-original-commit: 2f1d8d6
@robodoo
Copy link
Contributor

robodoo commented Sep 29, 2022

@fw-bot
Copy link
Contributor Author

fw-bot commented Sep 29, 2022

This PR targets 14.0 and is part of the forward-port chain. Further PRs will be created up to master.

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Sep 29, 2022
@robodoo robodoo added the forwardport This PR was created by @fw-bot label Sep 29, 2022
robodoo pushed a commit that referenced this pull request Sep 30, 2022
Steps to reproduce:

1. change system and admin's time zone to one with a different
date than UTC
2. enable multi-currency in the accounting module
3. add rates for the current date in local time zone and UTC
4. go to settings > currencies
5. the rate for the UTC date is displayed

To fix this, we should use a time zone aware field in `res_currency`.

opw-2945108

closes #101640

X-original-commit: 2f1d8d6
Signed-off-by: William André (wan) <wan@odoo.com>
Signed-off-by: Khanalizadeh Ahmad (khah) <khah@odoo.com>
@robodoo robodoo temporarily deployed to merge September 30, 2022 08:09 Inactive
@robodoo robodoo closed this Sep 30, 2022
@fw-bot fw-bot deleted the 14.0-13.0-fix-currency-rate-display-field-khah-PUgr-fw branch October 14, 2022 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
forwardport This PR was created by @fw-bot OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants