Skip to content

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

Merged
oriolgual merged 12 commits intomasterfrom
fix/4765-participatory_texts_error_uploading_utf-8_files
Jan 29, 2019
Merged

Fix/4765 participatory texts error uploading utf 8 files#4788
oriolgual merged 12 commits intomasterfrom
fix/4765-participatory_texts_error_uploading_utf-8_files

Conversation

@tramuntanal
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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