Skip to content
This repository has been archived by the owner on May 10, 2023. It is now read-only.

Disallow emojis for Thai sentences #431

Merged
merged 7 commits into from
Apr 17, 2021
Merged

Disallow emojis for Thai sentences #431

merged 7 commits into from
Apr 17, 2021

Conversation

bact
Copy link
Contributor

@bact bact commented Apr 17, 2021

  • Validation will not allow emojis
  • Cleanup will remove emojis

Background

Since mid-April 2021, Mozilla Thailand Community started to add more sentences via the Sentence Collector web interface (I believe this is a preparation towards their localization event on 25 April).

One of the thing few of the members do is exporting their own Twitter archive and add them to Sentence Collector.

@wannaphong has noticed that some of the sentences have emojis. So we need to reject or remove these emojis.

@bact bact changed the title Disallow emojis Disallow emojis for Thai sentences Apr 17, 2021
@MichaelKohler
Copy link
Member

If it helps you can run npm run lint locally in the server folder

@bact
Copy link
Contributor Author

bact commented Apr 17, 2021

If it helps you can run npm run lint locally in the server folder

Thanks Michael. I should go back to my another machine that has npm environment.

Copy link
Member

@MichaelKohler MichaelKohler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@MichaelKohler MichaelKohler merged commit a33a6e2 into common-voice:main Apr 17, 2021
MichaelKohler pushed a commit that referenced this pull request Apr 17, 2021
## [2.3.5](v2.3.4...v2.3.5) (2021-04-17)

### Bug Fixes

* disallow emojis for Thai sentences ([#431](#431)) ([a33a6e2](a33a6e2))
* set lang attribute on textarea ([78bf3c0](78bf3c0))
* set lang, dir=auto and translate=no on sentence elements (fixes [#428](#428)) ([54e2831](54e2831))
@MichaelKohler
Copy link
Member

🎉 This PR is included in version 2.3.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants