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

Updated Translations for "Never Opened" and "Last Opened" #696

Closed
wants to merge 22 commits into from

Conversation

FareesHussain
Copy link

@FareesHussain FareesHussain commented Nov 12, 2020

Related issue : #548
Related PR : #681

Explanation

In this pr, I've added the missing string resources and resolved string resource errors

Checklist

  • resolved string resource id ("language_te") error in string.xml-te
  • resolved string resource id ("message_duplicate_file_upload_body error
  • added string resource for "Never Opened" (in all languages)
  • added string resource for "Last Opened" (in all languages)

@Prakhar-Agarwal-byte
Copy link
Contributor

@FareesHussain Your branch has conflicts with amahi:beta branch. You must update your branch with new commits from beta, resolve the conflicts, and then send a PR with the updated branch.

� 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
@FareesHussain
Copy link
Author

Thanks @Prakhar-Agarwal-byte, updated

Copy link
Contributor

@Prakhar-Agarwal-byte Prakhar-Agarwal-byte left a 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.

@FareesHussain
Copy link
Author

@Prakhar-Agarwal-byte Thanks, I've updated it

Prakhar-Agarwal-byte
Prakhar-Agarwal-byte approved these changes Dec 23, 2020
Copy link
Contributor

@Prakhar-Agarwal-byte Prakhar-Agarwal-byte left a 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.

Copy link
Member

@chirag-jn chirag-jn left a 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.

@FareesHussain
Copy link
Author

I reformatted the changed files so that they match the coding conventions. Rest looks good.

Thanks a lot

@FareesHussain
Copy link
Author

@cpg Please have a look

@cpg
Copy link
Member

cpg commented Feb 27, 2021

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
@FareesHussain
Copy link
Author

FareesHussain commented Feb 27, 2021

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?

@cpg Done, Please have a look.

@chirag-jn
Copy link
Member

Please see the compilation issue @FareesHussain. In my opinion, it must have failed locally too.

<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>
Copy link
Member

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.

Copy link
Member

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.

Copy link
Author

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.

Copy link
Member

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.

Copy link
Author

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.

Copy link
Author

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

chirag-jn and others added 8 commits February 28, 2021 00:35
# 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
@github-actions
Copy link

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

Comment on lines +176 to +177
<string name="never_opened">Never Opened</string>
<string name="last_opened">Last Opened</string>
Copy link
Member

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.

Copy link
Author

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

Copy link
Author

Choose a reason for hiding this comment

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

Done please check

@chirag-jn chirag-jn changed the title Updated Translations Updated Translations for "Never Opened" and "Last Opened" Mar 1, 2021
@github-actions
Copy link

github-actions bot commented Mar 4, 2021

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

@cpg
Copy link
Member

cpg commented Mar 5, 2021

Did you guys get a chance to test this PR?
If so, how's it looking?

@github-actions
Copy link

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants