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

Fix/4765 participatory texts error uploading utf 8 files #4788

Merged

Conversation

tramuntanal
Copy link
Contributor

@tramuntanal tramuntanal commented Jan 24, 2019

🎩 What? Why?

In Participatory texts, do not crash when uploading plain text files from unknown encodings.

📌 Related Issues

📋 Subtasks

  • Add CHANGELOG entry
  • [-] Add documentation regarding the feature
  • [-] Add/modify seeds
  • Add tests
  • Add charlock_holmes gem.
  • New Decidim::IoEncoding in decidim-core.
  • Apply Decidim::IoEncoding.to_standard_encoding to incoming Markdown files.

Screenshots

Uploading file with accents and non ascii symbols is now ok.
decidimparticipatory_textsevol

The rendering in the last paragraph:
participatory_text-evols

@ghost ghost assigned tramuntanal Jan 24, 2019
@ghost ghost added the status: WIP label Jan 24, 2019
@tramuntanal
Copy link
Contributor Author

Hi @decidim/lot-core do you have any hint on why CodeClimate is failing due to "We had trouble running the grep engine."?

@oriolgual
Copy link
Contributor

Hi @decidim/lot-core do you have any hint on why CodeClimate is failing due to "We had trouble running the grep engine."?

Maybe it's related to decidim-dev/lib/decidim/dev/assets/iso-8859-15.md but I don't really know, you could try adding this file to the ignore list at CC.

@oriolgual
Copy link
Contributor

Could you also review de conflicts please?

@oriolgual oriolgual merged commit 46bf428 into master Jan 29, 2019
@ghost ghost removed the status: WIP label Jan 29, 2019
@tramuntanal tramuntanal deleted the fix/4765-participatory_texts_error_uploading_utf-8_files branch January 29, 2019 14:47
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.

2 participants