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 Indonesian translation #369

Merged
merged 4 commits into from
Oct 14, 2020
Merged

Conversation

imanudinsh
Copy link

Changes

Add Indonesian translation

Testing

  • Tested on a physical device
  • Added or modified unit test cases

Others

@codecov
Copy link

codecov bot commented Oct 14, 2020

Codecov Report

Merging #369 into master will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #369      +/-   ##
==========================================
+ Coverage   86.93%   86.96%   +0.02%     
==========================================
  Files          30       30              
  Lines        1623     1626       +3     
  Branches      178      178              
==========================================
+ Hits         1411     1414       +3     
  Misses        114      114              
  Partials       98       98              
Flag Coverage Δ
#android 86.25% <100.00%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...thub/ashutoshgngwr/noice/fragment/AboutFragment.kt 99.17% <100.00%> (+0.02%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8864438...8814d7f. Read the comment docs.

Copy link
Member

@ashutoshgngwr ashutoshgngwr left a comment

Choose a reason for hiding this comment

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

Hi @imanudinsh

Thank you for taking the time to translate Noice to Indonesian. I have requested a few changes. Please have a look. In addition to those changes, if you could also remove the region codes from all file paths. On Android, it will work but is not required since Indonesian is only spoken in Indonesia. In Fastlane metadata, it will not work since Google Play only recognises id tag. So to sum it up,

  • Rename values-in-rID to in
  • Rename fastlane/metadata/android/in-ID to fastlane/metadata/android/id

@@ -0,0 +1 @@
Santai, tingkatkan fokus, dan tingkatkan produktivitas dengan derau pelataran minimal.
Copy link
Member

Choose a reason for hiding this comment

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

The short description needs to be 80 characters at most. It's 86 right now. Can you please rephrase it to be <= 80?

<string name="preset_already_exists">Preset dengan nama yang diberikan sudah ada!</string>
<string name="preset_saved">Preset disimpan!</string>
<string name="saved_presets">Preset Tersimpan</string>
<string name="preset_delete_confirmation">Konfirmasi hapus% s?</string>
Copy link
Member

Choose a reason for hiding this comment

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

There's a typo here.

Suggested change
<string name="preset_delete_confirmation">Konfirmasi hapus% s?</string>
<string name="preset_delete_confirmation">Konfirmasi hapus %s?</string>

pengatur waktu.
</string>
<string name="reset">Setel ulang</string>
<string name="auto_sleep_schedule_cancelled">Pengatur waktu tidur otomatis dibatalkan!</string>
Copy link
Member

Choose a reason for hiding this comment

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

Google translates it to Sleep timer automatically canceled! but it needs to imply Sleep timer cancelled! or Automatic sleep timer cancelled!

@imanudinsh
Copy link
Author

Hello @ashutoshgngwr,

Thanks for reviewing my pull request, I'll fix it and come back asap

@imanudinsh
Copy link
Author

Hi @ashutoshgngwr,

I've fixed typos, updated translations, and changed the region code on all file paths.
please let me know if it still needs improvement.

Copy link
Member

@ashutoshgngwr ashutoshgngwr left a comment

Choose a reason for hiding this comment

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

Thanks, @imanudinsh! This is great work!

@ashutoshgngwr ashutoshgngwr merged commit 10f7a4b into trynoice:master Oct 14, 2020
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