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

Non-latin alphabets #130

Open
leilaz opened this issue Sep 1, 2018 · 3 comments · May be fixed by #194
Open

Non-latin alphabets #130

leilaz opened this issue Sep 1, 2018 · 3 comments · May be fixed by #194
Labels

Comments

@leilaz
Copy link
Collaborator

leilaz commented Sep 1, 2018

From Alexmar983 (and I have experienced this issue in the past as well):

The tool does not import decently namefile in Arabic or Greek on xls, in some countries I have a problem to understand the finalist... i have to ask the jurors what they vote to be sure.

@slaporte
Copy link
Member

slaporte commented Oct 4, 2018

Just to confirm, is this a problem with importing files or exporting votes?

@leilaz
Copy link
Collaborator Author

leilaz commented Oct 5, 2018

at the time of exporting.

@lgelauff
Copy link
Collaborator

The exported files are csv. It looks like excel recognizes these in the wrong encoding, and that is producing this bad user experience.

It looks like adding a BOM can help excel recognize the file as UTF-8. It seems this may fix it, but I have not been able to test it.

There is an example here that suggests how to add a BOM to the csvwriter. I'll submit a PR, but haven't properly set up the platform to test locally.

lgelauff added a commit that referenced this issue May 29, 2021
Should resolve #130 . Haven't been able to test locally though, so please tread with caution. Reusing from [example here](https://www.programcreek.com/python/example/98129/unicodecsv.DictWriter)
@lgelauff lgelauff linked a pull request May 29, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants