-
Notifications
You must be signed in to change notification settings - Fork 275
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
Updated Translations for "Never Opened" and "Last Opened" #696
Conversation
message_duplicate_file_upload_body language_te
@FareesHussain Your branch has conflicts with |
� Conflicts: � src/main/res/values-bn/strings.xml � src/main/res/values-es/strings.xml � src/main/res/values-te/strings.xml � src/main/res/values/strings.xml
Thanks @Prakhar-Agarwal-byte, updated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FareesHussain can you please add translations of the 2 new strings in German and Hindi as well so that your PR will be complete.
@Prakhar-Agarwal-byte Thanks, I've updated it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cpg I think this PR is working fine now.
src/main/java/org/amahi/anywhere/fragment/AlertDialogFragment.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reformatted the changed files so that they match the coding conventions. Rest looks good.
Thanks a lot |
@cpg Please have a look |
Oh no. Prior PR merges caused two conflicts. They look easy to merge, however I am not sure I will do it properly. Can you do it? |
� Conflicts: � src/main/res/values-te/strings.xml � src/main/res/values/strings.xml
@cpg Done, Please have a look. |
Please see the compilation issue @FareesHussain. In my opinion, it must have failed locally too. |
src/main/res/values-te/strings.xml
Outdated
<string name="language_fr">ఫ్రెంచ్</string> | ||
<string name="language_es">స్పానిష్</string> | ||
<string name="language_bn">బెంగాలీ</string> | ||
<string name="language_de">జర్మన్</string> | ||
<string name="header_settings">సెట్టింగులు</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
header_settings
is present twice.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed the issue. Let's see CI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks I've fixed the other conflicts too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CI still fails. Try building the app locally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I've tried it There is some issue with punjabi language. I'm unable to figure it out.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As my changes were initially minor shall I create a new PR for this
# Conflicts: # src/main/res/values-te/strings.xml
� Conflicts: � src/main/res/values-bn/strings.xml � src/main/res/values-es/strings.xml � src/main/res/values-fr/strings.xml � src/main/res/values-te/strings.xml
This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged. |
<string name="never_opened">Never Opened</string> | ||
<string name="last_opened">Last Opened</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add never_opened
and last_opened
for values-zh
(Chinese) too. Rest seems fine.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the delay. I'll do it now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done please check
This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged. |
Did you guys get a chance to test this PR? |
This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged. |
Related issue : #548
Related PR : #681
Explanation
In this pr, I've added the missing string resources and resolved string resource errors
Checklist