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 german strings and credits #138

Merged
merged 7 commits into from
Apr 22, 2020
Merged

Add german strings and credits #138

merged 7 commits into from
Apr 22, 2020

Conversation

schulle4u
Copy link
Contributor

@schulle4u schulle4u commented Apr 21, 2020

Changes

This adds german string ressources and translator credits. Unfortunately no metadata at this time, as I don't have a sufficient development/testing environment.

Connects #108

@codecov
Copy link

codecov bot commented Apr 21, 2020

Codecov Report

Merging #138 into master will increase coverage by 0.11%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #138      +/-   ##
============================================
+ Coverage     78.51%   78.63%   +0.11%     
  Complexity      107      107              
============================================
  Files            11       11              
  Lines           726      730       +4     
  Branches         85       85              
============================================
+ Hits            570      574       +4     
  Misses           88       88              
  Partials         68       68              
Impacted Files Coverage Δ Complexity Δ
...thub/ashutoshgngwr/noice/fragment/AboutFragment.kt 98.78% <100.00%> (+0.06%) 6.00 <0.00> (ø)

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 356dfff...e348ec0. 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 Steffen,

Thank you for taking the time to submit the translations. Can you please remove all the strings attributed with translatable="false" from values-de/strings.xml. Everything else looks good. I'll pull this in once these strings are removed.

Also, if you'd like to add metadata translations, just copy hi-IN directory as de-DE and fill in all the files that are present in the directory (translating from the en-US version). You don't need any test setup, let me worry about that. If you do decide to add metadata translations, please include screenshots as well.

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.

I am really sorry about being picky 😅 I also left some details about metadata translations in my last comment, if you're interested, please take a look.

<string name="app_website" translatable="false">https://ashutoshgngwr.github.io/noice</string>
<string name="app_playstore" translatable="false">com.github.ashutoshgngwr.noice</string>
<string name="app_github" translatable="false">ashutoshgngwr/noice</string>
<string name="connect_with_author">Autor kontaktieren</string>
Copy link
Member

@ashutoshgngwr ashutoshgngwr Apr 21, 2020

Choose a reason for hiding this comment

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

I think you deleted 49th line by mistake.

@schulle4u
Copy link
Contributor Author

Sorry, seems this is not may day 😀 Hope it is correct now.

Regarding playstore metadata, I currently cannot use Android Studio to build my localiced version in order to provide screenshots. I'm a blind user, and The software is partly inaccessible to the screen reader software which I'm using. So I'm doing the translation by hand.

@ashutoshgngwr
Copy link
Member

@schulle4u This looks good. I'd really appreciate if you can add just the translations for the metadata. I can add screenshots myself. Totally upto you, just let me know.

@schulle4u
Copy link
Contributor Author

OK thanks. Give me some minutes, I'm on my way.

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 @schulle4u
It needs some changes but I'll take it from here.

@ashutoshgngwr ashutoshgngwr merged commit 01214e9 into trynoice:master Apr 22, 2020
@schulle4u schulle4u deleted the german branch April 22, 2020 08:13
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