[ADD] currency_rate_update_fta: Currency Rate Update: Federal Tax Administration #750
Conversation
michelerusti
force-pushed
the
16.0-add-module-currency_rate_update_fta
branch
2 times, most recently
from
760ead2 to
d52f5b2
Compare
|
@michelerusti Thanks for your contribution! Can you add some tests to your module to please? |
michelerusti
force-pushed
the
16.0-add-module-currency_rate_update_fta
branch
6 times, most recently
from
ddd0561 to
31a4ebc
Compare
michelerusti
force-pushed
the
16.0-add-module-currency_rate_update_fta
branch
4 times, most recently
from
0e46db6 to
f3006a5
Compare
Hi, can you take a look at this PR please? |
| "installable": True, | ||
| "application": False, | ||
| "auto_install": False, | ||
| "price": 10.00, |
michelerusti
force-pushed
the
16.0-add-module-currency_rate_update_fta
branch
from
f3006a5 to
7853211
Compare
There was a problem hiding this comment.
Hello !
Thank you for this module, I tested it but encounter two issues.
The first one is you have to define _get_supported_currencies() in the provider otherwise you cannot choose any currency in the configuration of the provider.
The other is that I had to inverse the rate to get correct result. The https://www.rates.bazg.admin.ch/ is returning SELL rates (inverse_company_rate in Odoo).
You can see it in this commit
Feel free to cherry pick it or copy/paste it. I fetched the supported currencies live but it might be better to have a hard coded list? (it's what other providers do)
5 participants
No description provided.