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

Add validator for Lao (lo) #651

Merged
merged 6 commits into from Dec 1, 2022
Merged

Conversation

wannaphong
Copy link
Contributor

Propose code for Lao language validator.

It's basic validator code.

fn: (sentence) => {
return sentence.length < MIN_LENGTH || sentence.length > MAX_LENGTH;
},
error: `Number of characters must be between ${MIN_LENGTH} and ${MAX_LENGTH} (inclusive)`,
Copy link
Member

Choose a reason for hiding this comment

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

As with other languages, I think it would make sense to translate these error messages into Lao. What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As with other languages, I think it would make sense to translate these error messages into Lao. What do you think?

I agree with you. I will translate these messages into Lao.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

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.

Thank you for the contribution!

@MichaelKohler MichaelKohler merged commit f56d8a0 into common-voice:main Dec 1, 2022
MichaelKohler pushed a commit that referenced this pull request Dec 1, 2022
# [2.19.0](v2.18.2...v2.19.0) (2022-12-01)

### Bug Fixes

* remove Turkish sentences from a specific source (fixes [#650](#650)) ([#652](#652)) ([34fd2be](34fd2be))

### Features

* add validator for Lao (lo) ([#651](#651)) ([f56d8a0](f56d8a0))
@MichaelKohler
Copy link
Member

🎉 This PR is included in version 2.19.0 🎉

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.

None yet

2 participants