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

Added Icelandic, Latvian, Malayalam to language selector #9179

Merged
merged 3 commits into from
Oct 27, 2022

Conversation

HotDryNoodle-WH
Copy link
Contributor

@HotDryNoodle-WH HotDryNoodle-WH commented Oct 23, 2022

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

I downloaded newpipe-strings-is from NewPipe weblate. I created directory(values-is) in res. I added strings (newpipe-strings-is) to values-is. I added Icelandic, Latvian, Malayalam to language selector.

Before/After Screenshots/Screen Record

Before After 1 After 2
Before After After

Fixes the following issue(s)

APK testing

The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR.

Due diligence

@TobiGr TobiGr added the localisation / translation Everything that has to do with translations or Weblate label Oct 24, 2022
Copy link
Member

@TobiGr TobiGr 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 working on this.

Please do not upload translation files via GitHub. The changes need to be pulled from the weblate repository. I suggest to sync all translations in this step to prevent conflicts.
To add the weblate repo to your local git repo, run

git remote add weblate https://hosted.weblate.org/git/newpipe/strings/

reset your feature branch to weblate's dev and apply the changes to settings-key.xml. Commit and force push the changes to this branch.

app/src/main/res/values/settings_keys.xml Outdated Show resolved Hide resolved
@HotDryNoodle-WH
Copy link
Contributor Author

Thanks to your guidance. I completed the task and deleted the files I had previously added manually. I have tested the language selector in alphabetical order and everything works fine. Please feel free to let me know if there are any more problems?

Copy link
Member

@Stypox Stypox left a comment

Choose a reason for hiding this comment

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

Looks good to me, but let's wait for @TobiGr ;-)

@TobiGr TobiGr self-assigned this Oct 27, 2022
weblate and others added 3 commits October 27, 2022 20:25
Currently translated at 100.0% (645 of 645 strings)

Translated using Weblate (French)

Currently translated at 100.0% (645 of 645 strings)

Translated using Weblate (Greek)

Currently translated at 100.0% (645 of 645 strings)

Translated using Weblate (Greek)

Currently translated at 100.0% (645 of 645 strings)

Translated using Weblate (French)

Currently translated at 99.5% (642 of 645 strings)

Translated using Weblate (Greek)

Currently translated at 99.6% (643 of 645 strings)

Translated using Weblate (Icelandic)

Currently translated at 99.0% (636 of 642 strings)

Translated using Weblate (Polish)

Currently translated at 59.1% (42 of 71 strings)

Translated using Weblate (Lithuanian)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Japanese)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Icelandic)

Currently translated at 83.1% (534 of 642 strings)

Translated using Weblate (Punjabi (Pakistan))

Currently translated at 18.3% (118 of 642 strings)

Translated using Weblate (Punjabi (Pakistan))

Currently translated at 1.4% (1 of 71 strings)

Translated using Weblate (Icelandic)

Currently translated at 83.0% (533 of 642 strings)

Translated using Weblate (Aymara (Southern))

Currently translated at 0.1% (1 of 642 strings)

Translated using Weblate (Swedish)

Currently translated at 60.5% (43 of 71 strings)

Translated using Weblate (Russian)

Currently translated at 30.9% (22 of 71 strings)

Translated using Weblate (Lithuanian)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Swedish)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Turkish)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (French)

Currently translated at 100.0% (642 of 642 strings)

Added translation using Weblate (Punjabi (Pakistan))

Translated using Weblate (Icelandic)

Currently translated at 70.0% (450 of 642 strings)

Translated using Weblate (Undetermined)

Currently translated at 21.4% (138 of 642 strings)

Translated using Weblate (Azerbaijani)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Undetermined)

Currently translated at 18.3% (118 of 642 strings)

Translated using Weblate (Icelandic)

Currently translated at 2.8% (2 of 71 strings)

Translated using Weblate (Icelandic)

Currently translated at 51.5% (331 of 642 strings)

Translated using Weblate (French)

Currently translated at 91.5% (65 of 71 strings)

Translated using Weblate (Spanish)

Currently translated at 100.0% (71 of 71 strings)

Translated using Weblate (Italian)

Currently translated at 43.6% (31 of 71 strings)

Translated using Weblate (Basque)

Currently translated at 45.0% (32 of 71 strings)

Translated using Weblate (Punjabi)

Currently translated at 84.7% (544 of 642 strings)

Translated using Weblate (Azerbaijani)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Tamil)

Currently translated at 54.8% (352 of 642 strings)

Translated using Weblate (Tamil)

Currently translated at 54.8% (352 of 642 strings)

Translated using Weblate (Turkish)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Indonesian)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Czech)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Portuguese)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Korean)

Currently translated at 100.0% (642 of 642 strings)

Translated using Weblate (Spanish)

Currently translated at 100.0% (642 of 642 strings)

Added translation using Weblate (Undetermined)

Translated using Weblate (Icelandic)

Currently translated at 7.3% (47 of 642 strings)

Translated using Weblate (Icelandic)

Currently translated at 4.6% (30 of 642 strings)

Added translation using Weblate (Icelandic)

Co-authored-by: Alex25820 <alexs25820@gmail.com>
Co-authored-by: Chitraarasu <chitraarasu@kirshi.co>
Co-authored-by: Chitraarasu.k <kchitraarasu@gmail.com>
Co-authored-by: Fjuro <ifjuro@proton.me>
Co-authored-by: Florian <flo.site@zaclys.net>
Co-authored-by: Gediminas Murauskas <muziejusinfo@gmail.com>
Co-authored-by: GnuPGを使うべきだ <dieeeazpnnqbpddh@cock.email>
Co-authored-by: Gontzal Manuel Pujana Onaindia <thadahdenyse@gmail.com>
Co-authored-by: Hoseok Seo <ddinghoya@gmail.com>
Co-authored-by: Hosted Weblate <hosted@weblate.org>
Co-authored-by: Linerly <linerly@protonmail.com>
Co-authored-by: Mehmet Ali <2045uuttb@relay.firefox.com>
Co-authored-by: Nizami <nizamismidov4@gmail.com>
Co-authored-by: OneGuitars <xiaozhuojun1125@gmail.com>
Co-authored-by: Retrial <giwrgosmant@gmail.com>
Co-authored-by: SC <lalocas@protonmail.com>
Co-authored-by: Translator <kvb@tuta.io>
Co-authored-by: Trendyne <eiko@chiru.no>
Co-authored-by: Vasilis K <skyhirules@gmail.com>
Co-authored-by: atilluF <atilluf@outlook.com>
Co-authored-by: bgo-eiu <huyaqoob+toolforge@gmail.com>
Co-authored-by: gallegonovato <fran-carro@hotmail.es>
Co-authored-by: julboudin <boudin.julie@orange.fr>
Co-authored-by: rehork <cooky@e.email>
Co-authored-by: Артём Нефедов <artem10397g@gmail.com>
Translate-URL: https://hosted.weblate.org/projects/newpipe/metadata/es/
Translate-URL: https://hosted.weblate.org/projects/newpipe/metadata/eu/
Translate-URL: https://hosted.weblate.org/projects/newpipe/metadata/fr/
Translate-URL: https://hosted.weblate.org/projects/newpipe/metadata/is/
Translate-URL: https://hosted.weblate.org/projects/newpipe/metadata/it/
Translate-URL: https://hosted.weblate.org/projects/newpipe/metadata/pa_PK/
Translate-URL: https://hosted.weblate.org/projects/newpipe/metadata/pl/
Translate-URL: https://hosted.weblate.org/projects/newpipe/metadata/ru/
Translate-URL: https://hosted.weblate.org/projects/newpipe/metadata/sv/
Translation: NewPipe/Metadata
Copy link
Member

@TobiGr TobiGr 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.

I committed the pending changes in weblate, rebased the PR on weblate/dev and removed some of the unneeded commits in the process.

@sonarcloud
Copy link

sonarcloud bot commented Oct 27, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@TobiGr TobiGr merged commit 2225933 into TeamNewPipe:dev Oct 27, 2022
@TobiGr TobiGr mentioned this pull request Oct 27, 2022
@TobiGr
Copy link
Member

TobiGr commented Oct 27, 2022

@OneGuitars Just for internal tracking: was your PR part of the ANU SE course?

@HotDryNoodle-WH
Copy link
Contributor Author

@OneGuitars Just for internal tracking: was your PR part of the ANU SE course?

Yes, I am taking this course.

This was referenced Nov 4, 2022
@TobiGr TobiGr mentioned this pull request Nov 13, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
localisation / translation Everything that has to do with translations or Weblate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants