Skip to content
This repository has been archived by the owner on Nov 5, 2024. It is now read-only.

CSV export broken on locales using comma as decimal separator #520

Closed
Soukyuu opened this issue Feb 13, 2022 · 4 comments · Fixed by #2517
Closed

CSV export broken on locales using comma as decimal separator #520

Soukyuu opened this issue Feb 13, 2022 · 4 comments · Fixed by #2517
Labels
bug Something isn't working good first issue Good for newcomers help wanted Help us! Appreciated

Comments

@Soukyuu
Copy link

Soukyuu commented Feb 13, 2022

So I've just ran into this bug on my new device which I don't want to root, hence no way to transfer all the data from an offline account:

Exporting the CSV in i.e. German locale will mess up all the figures. 118,59€ turns into 11859,00€ etc.
As a workaround, you can change the phone language to English, restart the app and run the export.

Importing does not seem to be affected by locale, so importing a CSV exported in English locale on a device in German locale imports the figures correctly.
@ILIYANGERMANOV
Copy link
Collaborator

Thsnks for reporting!

@ILIYANGERMANOV ILIYANGERMANOV added bug Something isn't working good first issue Good for newcomers help wanted Help us! Appreciated labels Feb 13, 2022
@ILIYANGERMANOV
Copy link
Collaborator

Must be fixed in prod branch

@Tiihi
Copy link
Contributor

Tiihi commented Aug 28, 2023

I'm on it

@ILIYANGERMANOV
Copy link
Collaborator

Nice!! Make sure to send PR to prod and base your PR on prod

This was referenced Aug 28, 2023
Closed
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working good first issue Good for newcomers help wanted Help us! Appreciated
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Issue 520 Tiihi/ivy-wallet
Fix issue 520
3 participants
@Soukyuu @ILIYANGERMANOV @Tiihi