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

Add Bahasa Indonesia to supported languages #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mitra42
Copy link
Contributor

@mitra42 mitra42 commented Jan 14, 2025

Added support for Bahasa Indonesia, but left it out of LANGUAGE_ALL to avoid code bloat if other languages added.

To use it, define for example LANGUAGE_EN LANGUAGE_ID to have English and Indonesian as options.

Added support for Bahasa Indonesia, but left it out of LANGUAGE_ALL to avoid code bloat if other languages added. 

To use it, define for example LANGUAGE_EN LANGUAGE_ID to have English and Indonesian as options.
@mitra42
Copy link
Contributor Author

mitra42 commented Jan 14, 2025

See Issue #46

@Juerd
Copy link
Owner

Juerd commented Jan 14, 2025 via email

@mitra42
Copy link
Contributor Author

mitra42 commented Jan 14, 2025

Fair enough - could I ask to accept the PR for now and I'll commit to getting a native speaker to check it when I'm in Indonesia in a few weeks for the field trials, and doing a supplemental PR.

I left this out of _ALL because in the README you said Note: due to storage constraints on microcontroller flash filesystems, only widely used natural languages will be included.) and I wasn't clear what you considered widely used, happy to see it included by default.

This would save me having to run off a forked repo until after the trials.

@mitra42
Copy link
Contributor Author

mitra42 commented Jan 31, 2025

Hi - sorry I haven't heard from you, totally understand if you don't have time to maintain or review pull releases, but I guess that means we'll have to fork the repo or rather probably merge it into ours. Its a good library - thanks for the dev, sorry that having merged it we wont' be able to contribute back.

@Juerd
Copy link
Owner

Juerd commented Jan 31, 2025 via email

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