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

🌐 Add Persian translation for docs/fa/docs/advanced/sub-applications.md #9692

Merged
merged 9 commits into from Jun 27, 2023

Conversation

mojtabapaso
Copy link
Contributor

I made a directory for the advanced section.
I translated and placed the sub-applications.md file inside it

@tiangolo
Copy link
Owner

📝 Docs preview for commit 6226e54 at: https://648da98878649d03c1a5bd47--fastapi.netlify.app

@tiangolo tiangolo changed the title Add Persian translation for docs/fa/docs/advanced/sub-applications.md 🌐 Add Persian translation for docs/fa/docs/advanced/sub-applications.md Jun 22, 2023
@tiangolo tiangolo added lang-all Translations awaiting-review lang-fa Persian translations labels Jun 22, 2023
Copy link
Contributor

@aminalaee aminalaee left a comment

Choose a reason for hiding this comment

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

Looks really good, just some minor comments.

@@ -0,0 +1,72 @@
# برنامه فرعی - نصب
Copy link
Contributor

Choose a reason for hiding this comment

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

دلیل خاصی داره کلمه «برنامه اصلی» رو انتخاب کردی؟ کلمه «زیربرنامه» ترجمه دقیق تری نیست؟

Copy link
Contributor Author

Choose a reason for hiding this comment

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

بله نظر شما درست تر است و آن را تغییر دادم.
ممنون از توجهات به این مطلب

docs/fa/docs/advanced/sub-applications.md Outdated Show resolved Hide resolved
Co-authored-by: Amin Alaee <mohammadamin.alaee@gmail.com>
@tiangolo
Copy link
Owner

📝 Docs preview for commit 14550db at: https://64948757c0798e57b6bd8e37--fastapi.netlify.app

Copy link
Contributor Author

@mojtabapaso mojtabapaso left a comment

Choose a reason for hiding this comment

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

A better translation for the word Sub Applications

@tiangolo
Copy link
Owner

📝 Docs preview for commit ebd5eb5 at: https://64948b3595eb4000865e04a4--fastapi.netlify.app

Copy link
Contributor

@aminalaee aminalaee left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link

@heysaeid heysaeid left a comment

Choose a reason for hiding this comment

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

According to the comment I made, not all words should be translated in the technical content, because they reduce the meaning, readability and understanding, but it was good overall. Thankful

## نصب کردن یک برنامه **FastAPI**

"Mounting" به معنای افزودن یک برنامه کاملاً مستقل در یک مسیر خاص است، که پس از آن به مدیریت همه چیز در آن مسیر، با
عملیات مسیر اعلام شده در آن زیر برنامه، رسیدگی می کند.

Choose a reason for hiding this comment

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

به نظرم تو ترجمه مطالب تکنیکال نباید تمام کلمات به فارسی ترجمه بشن چون معنی, خوانایی و درک جمله رو پایین میارن, برای مثال تو این قسمت استفاده از path operations به جای "عملیات مسیر" هم معنی بهتری داره و هم خواناتر هستش یا اینکه استفاده کلمه sub-application بهتر از کلمه "زیر برنامه" هستش, برای عنوان هم فکر میکنم اینجا کلمه اتصال بهتر از نصب باشه

Copy link
Contributor

Choose a reason for hiding this comment

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

I agree with the suggestion, but there’s a limit that you need to draw the line.
For example you are right that “path operations” should not be translated, but we can translate sub-application as we already have a good translation for application.

I checked the documentation translation and improved it a bit.
@tiangolo
Copy link
Owner

📝 Docs preview for commit 402960b at: https://64953d22f52eb444e494bdf9--fastapi.netlify.app

Copy link

@heysaeid heysaeid left a comment

Choose a reason for hiding this comment

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

Some suggestions to improve the translation, also the code blocks are not displayed
You can check in the following link:
https://64953d22f52eb444e494bdf9--fastapi.netlify.app/fa/advanced/sub-applications/


### اتصال زیر برنامه

در برنامه سطح بالا `app` اتصال زیر برنامه `subapi` در این نمونه `/subapi` در مسیر قرار میدهد و میشود:

Choose a reason for hiding this comment

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

در برنامه اصلی(سطح بالا) که app نام دارد، زیربرنامه subapi را متصل (mount) کنید.

در این مورد، زیربرنامه در مسیر /subapi متصل خواهد شد.

## اتصال (mount) به یک برنامه **FastAPI**

"Mounting" به معنای افزودن یک برنامه کاملاً مستقل در یک مسیر خاص است، که پس از آن مدیریت همه چیز در آن مسیر، با
path operations (عملیات مسیر) اعلام شده در آن زیر برنامه می باشد.

Choose a reason for hiding this comment

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

عملیات مسیر جالب نیستش پس بهتر نباشه یا حداقل «عملیات های مسیر» درست تر هستش


### اسناد API خودکار را بررسی کنید

حالا ‘uvicorn’ را برای برنامه اصلی اجرا کنید، برای مثال اگر فایل شما`main.py` هست:

Choose a reason for hiding this comment

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

برنامه را با استفاده از uvicorn اجرا کنید، اگر فایل شما main.py نام دارد، دستور زیر را وارد کنید

همچنین تو ترجمه ای که انجام دادی یک space برای main.py جا افتاده 😃

</div>

و باز کنید مستندات را در
آدرس <a href="http://127.0.0.1:8000/docs" class="external-link" target="_blank">http://127.0.0.1:8000/docs</a>

Choose a reason for hiding this comment

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

صفحه مستندات را در آدرس http://127.0.0.1:8000/docs باز کنید.


و باز کنید مستندات را در
آدرس <a href="http://127.0.0.1:8000/docs" class="external-link" target="_blank">http://127.0.0.1:8000/docs</a>
اسناد API خودکار برنامه اصلی را مشاهده خواهید کرد که فقط شامل عملیات مسیر خود می شود:

Choose a reason for hiding this comment

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

خب الان مشاهده میکنید که اسناد API خودکار برنامه اصلی فقط شامل path operations خود میباشد

این ترجمه و ترجمه بالا باید تو دو خط جدا باشند


### جرئیات فنی : `root_path`

هنگامی که یک زیر برنامه را همانطور که در بالا توضیح داده شد متصل می کنید,FastAPI با استفاده از مکانیزمی از مشخصات ASGI به نام `root_path` ارتباط مسیر mount را برای زیر برنامه انجام می دهد.

Choose a reason for hiding this comment

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

فاصله ها خوب رعایت نشدن

Make some changes to the translated documentation
@tiangolo
Copy link
Owner

📝 Docs preview for commit b7572cc at: https://64955d89e032e50b82130ef4--fastapi.netlify.app

Copy link

@heysaeid heysaeid left a comment

Choose a reason for hiding this comment

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

The code blocks are not correct (I think because of space it is), that's why nothing is loaded, they should be corrected

You can see it in the following link:
https://64955d89e032e50b82130ef4--fastapi.netlify.app/fa/advanced/sub-applications/

ابتدا برنامه اصلی سطح بالا، **FastAPI** و path operations آن را ایجاد کنید:

```Python hl_lines="3 6-8"
{!../../../ docs_src / sub_applications / tutorial001.py!}

Choose a reason for hiding this comment

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

{!../../../docs_src/sub_applications/tutorial001.py!}
باید اصلاح بشه


این زیر برنامه فقط یکی دیگر از برنامه های استاندارد FastAPI است، اما این برنامه ای است که متصل می شود:

```Python hl_lines="11 14-16"

Choose a reason for hiding this comment

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

{!../../../docs_src/sub_applications/tutorial001.py!}
باید اصلاح بشه


در برنامه سطح بالا `app` اتصال زیر برنامه `subapi` در این نمونه `/subapi` در مسیر قرار میدهد و میشود:

```Python hl_lines="11 19"

Choose a reason for hiding this comment

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

{!../../../docs_src/sub_applications/tutorial001.py!}
باید اصلاح بشه

There were some wrong spaces in the links, I deleted them.
@tiangolo
Copy link
Owner

📝 Docs preview for commit 22bcb14 at: https://64959158be371e2b9e7b689e--fastapi.netlify.app

Copy link

@heysaeid heysaeid left a comment

Choose a reason for hiding this comment

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

LGTM 😉

@tiangolo
Copy link
Owner

📝 Docs preview for commit cad7ac6 at: https://649a316c443f2f2085861512--fastapi.netlify.app

@tiangolo
Copy link
Owner

Great, thank you @mojtabapaso! 🎉

And thanks for the reviews @aminalaee and @heysaeid 🙇 ☕

@tiangolo tiangolo merged commit 6ba4492 into tiangolo:master Jun 27, 2023
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-2 lang-all Translations lang-fa Persian translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants