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

Support for bengali language #674

Closed
wants to merge 2 commits into from
Closed

Support for bengali language #674

wants to merge 2 commits into from

Conversation

Ayan-10
Copy link
Contributor

@Ayan-10 Ayan-10 commented Sep 25, 2020

Bengali{bn) new language added.
@chirag-jn As you said I add a new language Please review it.

@chirag-jn chirag-jn changed the title beta Localization support for bengali Sep 25, 2020
Comment on lines +1 to +8
/*___Generated_by_IDEA___*/

package org.amahi.anywhere;

/* This stub is only used by the IDE. It is NOT the BuildConfig class actually packed into the APK */
public final class BuildConfig {
public final static boolean DEBUG = Boolean.parseBoolean(null);
}
Copy link
Member

Choose a reason for hiding this comment

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

What's this file? Is it necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chirag-jn I don't know about this file. when i created 1st pull request i get a error in github then a went to git bush and commit all files and then i created pull request and github did not show me any eror. I just edited string.xml files nothing more than that.

Copy link
Member

Choose a reason for hiding this comment

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

No issues. Check if removing this file is sufficient and make another commit to the same branch.

<string name="message_empty_recent_files">সাম্প্রতিক ফাইল অনুপস্থিত</string>
<string name="message_empty_shares">শেয়ার অনুপস্থিত</string>
<string name="message_error_account_exists">অ্যাকাউন্ট ইতিপূর্বে আছে।</string>
<string name="message_error_amahi_anywhere_app">The Amahi Anywhere অ্যাপটি আপনার সার্ভার এর কোন জায়গার আয়ত্তাধীন নয়।দয়া করে The Amahi Anywhere অ্যাপটি আপনার নেটওয়ার্কে ইনস্টল, চলমান আর আয়ত্তাধীন তা নিশ্চিত করুন।</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 check the translation of this sentence if it's translated properly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chirag-jn yes this sentense is translated properly. I didn't changed The Amahi Anywhere because I thought it is a brnad name and It should be stay as it is. Should I aslo translate "The Amahi Anywhere"?

Copy link
Member

Choose a reason for hiding this comment

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

"The Amahi Anywhere" isn't a brand per se but "Amahi Anywhere" is

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok understand I will fix this @chirag-jn

<string name="message_notice_create_some">কিছু তৈরি করে পুনরায় চেষ্টা করুন</string>
<string name="message_notice_visit_some">কিছু পরিদর্শন করে পুনরায় চেষ্টা করুন</string>
<string name="message_offline_file_deleted">অফ্লাইন ফাইল সফলভাবে সরানো হয়েছে</string>
<string name="message_progress_file_downloading">ফাইলটি ডাউনলোড হচ্ছে ...</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 see these triple dots in other values files. Studio must give a warning for placing three dots like this.

Copy link
Contributor Author

@Ayan-10 Ayan-10 Sep 25, 2020

Choose a reason for hiding this comment

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

@chirag-jn No, Studio didn't give me any error and even in default language and other string values this three dots are like this.

Copy link
Member

Choose a reason for hiding this comment

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

In other files, it's represented as one character, but here it's represented as three characters. Also it will be a warning and not an error. Try running linter on this file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok. I am tring I will inform you after sometime

Comment on lines +1 to +7
/*___Generated_by_IDEA___*/

package org.amahi.anywhere;

/* This stub is only used by the IDE. It is NOT the Manifest class actually packed into the APK */
public final class Manifest {
}
Copy link
Member

Choose a reason for hiding this comment

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

What's this file? Is it necessary?

Comment on lines +1 to +7
/*___Generated_by_IDEA___*/

package org.amahi.anywhere;

/* This stub is only used by the IDE. It is NOT the R class actually packed into the APK */
public final class R {
}
Copy link
Member

Choose a reason for hiding this comment

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

What's this file? Is it necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chirag-jn I don't know about this file. when i created 1st pull request i get a error in github then a went to git bush and commit all files and then i created pull request and github did not show me any eror. I just edited string.xml files nothing more than that.

Copy link
Member

Choose a reason for hiding this comment

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

Please check the files again. We don't need these files.

@chirag-jn
Copy link
Member

Adding language file isn't enough. You have to add support for switching the language too! Check LANGUAGES.md file.

@chirag-jn chirag-jn added on hold Hold on there, buddy! ui/ux User Interface & User Experience Issues/Improvements labels Sep 25, 2020
@chirag-jn
Copy link
Member

This PR is a step towards #548

@chirag-jn chirag-jn changed the title Localization support for bengali Support for bengali language Sep 25, 2020
@cpg cpg closed this Sep 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on hold Hold on there, buddy! ui/ux User Interface & User Experience Issues/Improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants