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

Urdu lang #2509

Merged
merged 6 commits into from
Sep 20, 2022
Merged

Urdu lang #2509

merged 6 commits into from
Sep 20, 2022

Conversation

ArshErgon
Copy link
Contributor

added support for urdu language.

@PrestonN PrestonN enabled auto-merge (squash) August 26, 2022 05:07
@efb4f5ff-1298-471a-8973-3d47447115dc efb4f5ff-1298-471a-8973-3d47447115dc added the PR: waiting for review For PRs that are complete, tested, and ready for review label Aug 26, 2022
@ChunkyProgrammer
Copy link
Member

We suggest using https://hosted.weblate.org/engage/free-tube/ to add new languages

Search Filters:
Search Filters: 'فلٹرز تلاش کریں۔'
Sort By:
Sort By: Sort By
Copy link
Member

Choose a reason for hiding this comment

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

I think the non translated ones should be set to empty strings
Ex:

Suggested change
Sort By: Sort By
Sort By: ''

auto-merge was automatically disabled August 29, 2022 08:21

Head branch was pushed to by a user without write access

@PrestonN PrestonN enabled auto-merge (squash) August 29, 2022 08:21
Copy link
Contributor Author

@ArshErgon ArshErgon left a comment

Choose a reason for hiding this comment

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

re-edit the language

Copy link
Member

@ChunkyProgrammer ChunkyProgrammer left a comment

Choose a reason for hiding this comment

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

Two more english texts

Screenshot Success: 'اسکرین شاٹ کو بطور "$" محفوظ کیا گیا'
Screenshot Error: 'اسکرین شاٹ ناکام ہو گیا۔ $'

Yes: Yes
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Yes: Yes
Yes: ''

Screenshot Error: 'اسکرین شاٹ ناکام ہو گیا۔ $'

Yes: Yes
No: No
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
No: No
No: ''

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, its done, can you please tell me how can I be running the project locally? will npm start or yarn dev will work?

Copy link
Member

Choose a reason for hiding this comment

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

yarn install to install the dependencies and then yarn dev to run in dev mode.

Copy link
Member

Choose a reason for hiding this comment

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

What absidue said + if you're trying to test out your language, you'll need to add "ur" to

const activeLocales = ['en-US', 'en_GB', 'ar', 'bg', 'ca', 'cs', 'da', 'de-DE', 'el', 'es', 'es_AR', 'es-MX', 'et', 'eu', 'fi', 'fr-FR', 'gl', 'he', 'hu', 'hr', 'id', 'is', 'it', 'ja', 'ko', 'lt', 'nb_NO', 'nl', 'nn', 'pl', 'pt', 'pt-BR', 'pt-PT', 'ro', 'ru', 'sk', 'sl', 'sr', 'sv', 'tr', 'uk', 'vi', 'zh-CN', 'zh-TW']

auto-merge was automatically disabled August 29, 2022 12:05

Head branch was pushed to by a user without write access

@PrestonN PrestonN enabled auto-merge (squash) August 29, 2022 12:05
@ChunkyProgrammer
Copy link
Member

@ArshErgon will you be adding more translations or would you want this to be merged as is?

@ArshErgon
Copy link
Contributor Author

@ChunkyProgrammer yes, I will be adding more languages, but at now, not.

@efb4f5ff-1298-471a-8973-3d47447115dc

@ArshErgon more languages or more translations of this language

@ArshErgon
Copy link
Contributor Author

@efb4f5ff-1298-471a-8973-3d47447115dc I will be adding more languages.

@ChunkyProgrammer
Copy link
Member

tested, there are some issues with the yaml format. would you want me to push the commit to fix the yaml? @ArshErgon

Copy link
Member

@ChunkyProgrammer ChunkyProgrammer left a comment

Choose a reason for hiding this comment

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

Make the following changes and this PR should be ready to approve (you should also be able to test your changes once these changes are made to make the yaml valid)

Settings:
The app needs to restart for changes to take effect. Restart and apply change?: 'دی
تبدیلیاں اثر انداز ہونے کے لیے ایپ کو دوبارہ شروع کرنے کی ضرورت ہے۔ دوبارہ شروع کریں اور تبدیلی کا اطلاق کریں؟'
General Settings: 'ترتیبات'
Copy link
Member

Choose a reason for hiding this comment

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

This key should remain blank (General Settings is translated below)

Suggested change
General Settings: 'ترتیبات'
General Settings:

Playlists: ''
User Playlists:
Your Playlists: ''
Playlist Message: 'یہ صفحہ مکمل طور پر کام کرنے والی پلے لسٹس کا عکاس نہیں ہے۔ یہ صرف
Copy link
Member

Choose a reason for hiding this comment

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

remove outer 's as string contains one in it

Suggested change
Playlist Message: 'یہ صفحہ مکمل طور پر کام کرنے والی پلے لسٹس کا عکاس نہیں ہے۔ یہ صرف
Playlist Message: یہ صفحہ مکمل طور پر کام کرنے والی پلے لسٹس کا عکاس نہیں ہے۔ یہ صرف

Your Playlists: ''
Playlist Message: 'یہ صفحہ مکمل طور پر کام کرنے والی پلے لسٹس کا عکاس نہیں ہے۔ یہ صرف
ان ویڈیوز کی فہرست بناتا ہے جنہیں آپ نے محفوظ کیا ہے یا پسند کیا ہے۔ جب کام ختم ہو جائے تو سب
فی الحال یہاں موجود ویڈیوز کو 'پسندیدہ' پلے لسٹ میں منتقل کر دیا جائے گا۔'
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
فی الحال یہاں موجود ویڈیوز کو 'پسندیدہ' پلے لسٹ میں منتقل کر دیا جائے گا۔'
فی الحال یہاں موجود ویڈیوز کو 'پسندیدہ' پلے لسٹ میں منتقل کر دیا جائے گا۔

Current Invidious Instance: ''
# $ is replaced with the default Invidious instance
The currently set default instance is $: ''
No default instance has been set:''
Copy link
Member

Choose a reason for hiding this comment

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

a colon and a space(": ") should separate keys from values

Suggested change
No default instance has been set:''
No default instance has been set: ''

No Action: ''
Theme Settings:
Theme Settings: ''
Match Top Bar with Main Color:''
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Match Top Bar with Main Color:''
Match Top Bar with Main Color: ''

Expand Side Bar by Default: ''
Disable Smooth Scrolling: ''
UI Scale: ''
Hide Side Bar Labels:''
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Hide Side Bar Labels:''
Hide Side Bar Labels: ''

UI Scale: ''
Hide Side Bar Labels:''
Base Theme:
Base Theme:''
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Base Theme:''
Base Theme: ''

Ignore Warnings: 'انتباہات کو دبائیں جب موجودہ بیرونی کھلاڑی تعاون نہیں کرتا ہے۔
موجودہ کارروائی (مثلاً پلے لسٹ کو تبدیل کرنا، وغیرہ)۔'
Custom External Player Arguments:
'کوئی بھی حسب ضرورت کمانڈ لائن آرگومنٹس، جو سیمیکولنز (';') سے الگ کیے گئے ہیں،
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
'کوئی بھی حسب ضرورت کمانڈ لائن آرگومنٹس، جو سیمیکولنز (';') سے الگ کیے گئے ہیں،
کوئی بھی حسب ضرورت کمانڈ لائن آرگومنٹس، جو سیمیکولنز (';') سے الگ کیے گئے ہیں،

موجودہ کارروائی (مثلاً پلے لسٹ کو تبدیل کرنا، وغیرہ)۔'
Custom External Player Arguments:
'کوئی بھی حسب ضرورت کمانڈ لائن آرگومنٹس، جو سیمیکولنز (';') سے الگ کیے گئے ہیں،
آپ بیرونی کھلاڑی کو منتقل کرنا چاہتے ہیں۔'
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
آپ بیرونی کھلاڑی کو منتقل کرنا چاہتے ہیں۔'
آپ بیرونی کھلاڑی کو منتقل کرنا چاہتے ہیں۔

@ChunkyProgrammer ChunkyProgrammer added PR: changes requested and removed PR: waiting for review For PRs that are complete, tested, and ready for review labels Sep 9, 2022
auto-merge was automatically disabled September 12, 2022 03:29

Head branch was pushed to by a user without write access

@PrestonN PrestonN enabled auto-merge (squash) September 12, 2022 03:30
@ArshErgon ArshErgon requested review from absidue and ChunkyProgrammer and removed request for absidue September 12, 2022 03:35
@ChunkyProgrammer
Copy link
Member

ChunkyProgrammer commented Sep 20, 2022

As mentioned above, it would be a lot faster for future language contributions to be done through our weblate https://hosted.weblate.org/engage/free-tube/ . File is valid YAML so lgtm. Thanks so much for contributing to freetube, we hope to see more contributions in the future :)

@PrestonN PrestonN merged commit 60166eb into FreeTubeApp:development Sep 20, 2022
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Sep 20, 2022
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.

None yet

6 participants