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

Farsi Support 🇮🇷🇦🇫🇹🇯 #279

Merged
merged 5 commits into from
Mar 3, 2020
Merged

Farsi Support 🇮🇷🇦🇫🇹🇯 #279

merged 5 commits into from
Mar 3, 2020

Conversation

ashkan18
Copy link
Contributor

First attempt in Adding Farsi Language support.

Start of #278

cc: @orta

@msftclas
Copy link

msftclas commented Feb 24, 2020

CLA assistant check
All CLA requirements met.

@orta
Copy link
Contributor

orta commented Feb 24, 2020

All i8n details here: #100

Copy link
Contributor

@Ladsgroup Ladsgroup left a comment

Choose a reason for hiding this comment

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

I can also make a follow up if you want to. Thanks for adding the support!

// به محل بازی با تایپ اسکریپت خوش آمدید.
// اینجا ابزاری برای امتحان و آموزش تایپ اسکریپت در مرورگر شماست.
// این ابزار به شما این امکان را می دهد که به سادگی
// قابلیت های تایپ اسکریپت را آزمایش کنید و آموخته های خود را با دیگران در میان بگذارید.
Copy link
Contributor

Choose a reason for hiding this comment

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

"آموخته های" should be replaced with "آموخته‌های"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good call! still struggling with typing in Farsi 🙌

// اینجا ابزاری برای امتحان و آموزش تایپ اسکریپت در مرورگر شماست.
// این ابزار به شما این امکان را می دهد که به سادگی
// قابلیت های تایپ اسکریپت را آزمایش کنید و آموخته های خود را با دیگران در میان بگذارید.
// برای فرستادن آزمایش خود با دیگران تنها فرستادن آدرس صفحه آزمایش شما کافیست.
Copy link
Contributor

Choose a reason for hiding this comment

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

"فرستادن" means "sending", if you mean "sharing". Use "به اشتراک‌گذاری"


const hello = 'سلام'

// در سمت راست شما نتیحه مترجم تایپ اسکریپت را می توانید ببیید.
Copy link
Contributor

Choose a reason for hiding this comment

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

"ببیید" has a typo


// حالا که بیشتر با این ابزار آشنا شدیم می توانیم
// بیشتر به امکاناتی که تایپ اسکریپت برای راحتی کار با جاوااسکریپت در اختیار ما قرار میدهد آشنا شویم.
// در این قسمت تلاش می کنیم که تا حد امکان کدی شبیه به جاوااسکریپت
Copy link
Contributor

Choose a reason for hiding this comment

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

This line doesn't make much sense, is it left before finishing?

@@ -0,0 +1,3 @@
## مثال های جاوااسکریپت

این مثال ها به ما چگونگی اجرای جاوااسکریپت توسط تایپ اسکریپت را نشان می دهند. با اینکه در بعضی از مثال ها از تایپ قابلیت های ساده تایپ اسکریپت استفاده می کنیم اما هدف اصلی این دسته از مثال ها نشان دادن سادگی استفاده مجدد از دانسته های کاربر در جاوااسکریپت در تایپ اسکریپت است.
Copy link
Contributor

Choose a reason for hiding this comment

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

Use ZWNJ for TypeScript. According to Persian standards, you should not use space for those cases.

@@ -0,0 +1,3 @@
## مثال های جاوااسکریپت

این مثال ها به ما چگونگی اجرای جاوااسکریپت توسط تایپ اسکریپت را نشان می دهند. با اینکه در بعضی از مثال ها از تایپ قابلیت های ساده تایپ اسکریپت استفاده می کنیم اما هدف اصلی این دسته از مثال ها نشان دادن سادگی استفاده مجدد از دانسته های کاربر در جاوااسکریپت در تایپ اسکریپت است.
Copy link
Contributor

Choose a reason for hiding this comment

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

"دانسته های" -> use ZWNJ instead of space

@Ladsgroup
Copy link
Contributor

Hey, How can I help this move forward? This can be merged for now and I make a follow up for fixes. What do you think?

@ashkan18
Copy link
Contributor Author

ashkan18 commented Mar 3, 2020

Hi @Ladsgroup , sorry I've been in between switching jobs now and haven't had a chance to address your feedback, they all make sense to me, just need to find time and do it. Bu everything works for me, this is going to be an ongoing teamwork anyways. ❤️

@orta
Copy link
Contributor

orta commented Mar 3, 2020

Yep, merging this and improving in the future is fine with me 👍

@ashkan18 - is there a good day for our pairing session? I'm free weds-fri, and thanks to COVID19, I've got a completely empty two weeks of plans next week as I'm not out in redmond

@orta orta merged commit 0c8d217 into microsoft:v2 Mar 3, 2020
@ashkan18
Copy link
Contributor Author

ashkan18 commented Mar 3, 2020

Thanks @orta just came to address some comments 😄 i'll have time next week if that works for you, we can finalize it on slack but i'll be out of town from tomorrow till Sunday.

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.

4 participants