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

Update fa.json #1138

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update fa.json #1138

wants to merge 1 commit into from

Conversation

Clopas
Copy link
Contributor

@Clopas Clopas commented Sep 8, 2023

Sorry, I had to close the last PR.
I used ChatGPT to review my translation. It compared the en and fa texts for each line of text and didn't find any errors. The output is attached here. CHATGPT-REVIEW.md

To reproduce it:

1.Paste this query to Chatgpt and press enter:

I have two texts both use json and html formats. first one is the original which is in english. the second is translated in persian. Review the persian and compare it with the english one in terms of translation. I'm first pasting the english.

2.Paste the en.json content and press enter.

3.Paste the fa.json content and press enter.

*Added missing lines.
*Improved existing translations.
*Fully aligned with en.json file.
*Used phrases popular in persian crypto communities.
Copy link
Member

@jholdstock jholdstock left a comment

Choose a reason for hiding this comment

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

Technical review

All new strings are displayed OK on the site. All HTML elements embedded in the strings are proper. A few minor in-line suggestions made for extra \n new line characters which are unnecessary.

In order for Persian to be added to the language dropdown, please add the following segment to the bottom of src/config.yml:

    fa:
        contentDir: content/fa
        languageName: فارسی
        weight: 140
        params:
            languageDir: rtl

"downloads_app_commandline": "نصب کننده/آپدیت خودکار و چند پلتفورمی برای برنامه های خط فرمان.",
"downloads_app_dcrdex": "کیف پول سبک به همراه پشتیبانی از DEX و چند ارزی. فاقد استیکینگ یا قابلیت های کیف پول پیشرفته.",
"downloads_app_decrediton": "کیف پول رابط کاربری گرافیکی برای Windows، macOS و Linux.",
"downloads_app_exodus": "Exodus\n برنامه ای کاربرپسند است که قابلیت های ایمن سازی، مدیریت و تبادل دارایی های زنجیره بلوکی را در یک جا جمع می کند.",
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
"downloads_app_exodus": "Exodus\n برنامه ای کاربرپسند است که قابلیت های ایمن سازی، مدیریت و تبادل دارایی های زنجیره بلوکی را در یک جا جمع می کند.",
"downloads_app_exodus": "Exodus برنامه ای کاربرپسند است که قابلیت های ایمن سازی، مدیریت و تبادل دارایی های زنجیره بلوکی را در یک جا جمع می کند.",

"downloads_app_dcrdex": "کیف پول سبک به همراه پشتیبانی از DEX و چند ارزی. فاقد استیکینگ یا قابلیت های کیف پول پیشرفته.",
"downloads_app_decrediton": "کیف پول رابط کاربری گرافیکی برای Windows، macOS و Linux.",
"downloads_app_exodus": "Exodus\n برنامه ای کاربرپسند است که قابلیت های ایمن سازی، مدیریت و تبادل دارایی های زنجیره بلوکی را در یک جا جمع می کند.",
"downloads_app_guarda": "Guarda\n با کیف پول‌های چند ارزی مبتنی بر وب، رایانه، و دستگاه همراه ارائه می‌شود.",
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
"downloads_app_guarda": "Guarda\n با کیف پول‌های چند ارزی مبتنی بر وب، رایانه، و دستگاه همراه ارائه می‌شود.",
"downloads_app_guarda": "Guarda با کیف پول‌های چند ارزی مبتنی بر وب، رایانه، و دستگاه همراه ارائه می‌شود.",

"downloads_app_decrediton": "کیف پول رابط کاربری گرافیکی برای Windows، macOS و Linux.",
"downloads_app_exodus": "Exodus\n برنامه ای کاربرپسند است که قابلیت های ایمن سازی، مدیریت و تبادل دارایی های زنجیره بلوکی را در یک جا جمع می کند.",
"downloads_app_guarda": "Guarda\n با کیف پول‌های چند ارزی مبتنی بر وب، رایانه، و دستگاه همراه ارائه می‌شود.",
"downloads_app_ledger": "Ledger Live\n کیف پولی مبتنی بر وب است که با کیف پول سخت افزاری Ledger ایمن‌سازی شده است.",
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
"downloads_app_ledger": "Ledger Live\n کیف پولی مبتنی بر وب است که با کیف پول سخت افزاری Ledger ایمن‌سازی شده است.",
"downloads_app_ledger": "Ledger Live کیف پولی مبتنی بر وب است که با کیف پول سخت افزاری Ledger ایمن‌سازی شده است.",

"downloads_app_guarda": "Guarda\n با کیف پول‌های چند ارزی مبتنی بر وب، رایانه، و دستگاه همراه ارائه می‌شود.",
"downloads_app_ledger": "Ledger Live\n کیف پولی مبتنی بر وب است که با کیف پول سخت افزاری Ledger ایمن‌سازی شده است.",
"downloads_app_mobilewallets": "کیف پول های تلفن همراه دیکْرِد(Decred) موجود برای پلتفرم های اندروید و iOS",
"downloads_app_trezor": "Trezor\n یک کیف پول سخت افزاری است که توسط Decrediton and Exodus wallets پشتیبانی میشود.",
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
"downloads_app_trezor": "Trezor\n یک کیف پول سخت افزاری است که توسط Decrediton and Exodus wallets پشتیبانی میشود.",
"downloads_app_trezor": "Trezor یک کیف پول سخت افزاری است که توسط Decrediton and Exodus wallets پشتیبانی میشود.",

"downloads_app_ledger": "Ledger Live\n کیف پولی مبتنی بر وب است که با کیف پول سخت افزاری Ledger ایمن‌سازی شده است.",
"downloads_app_mobilewallets": "کیف پول های تلفن همراه دیکْرِد(Decred) موجود برای پلتفرم های اندروید و iOS",
"downloads_app_trezor": "Trezor\n یک کیف پول سخت افزاری است که توسط Decrediton and Exodus wallets پشتیبانی میشود.",
"downloads_app_trust": "Trust Wallet\nیک کیف پول چند ارزی و ایمن برای اندروید و iOS است.",
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
"downloads_app_trust": "Trust Wallet\nیک کیف پول چند ارزی و ایمن برای اندروید و iOS است.",
"downloads_app_trust": "Trust Wallet یک کیف پول چند ارزی و ایمن برای اندروید و iOS است.",

Copy link
Member

@jholdstock jholdstock left a comment

Choose a reason for hiding this comment

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

Language review

I am not an Persian speaker, but I ran the entire file through Google Translate and everything looks fine to me. I suggest this PR is good to merge after the suggested changes are made.

"downloads_category_core_software": "نرم‌افزار اصلی",
"downloads_category_hardware_wallets": "کیف پول های سخت افزاری",
"downloads_category_third_party_wallets": "کیف پول های شخص ثالث",
"downloads_description": "اگر تازه با دیکْرِد(Decred) آشنا شده‌اید و می‌خواهید اطلاعات عمیق‌تری داشته باشید، حتماً <a href='https://docs.decred.org' target='_blank' rel='noopener noreferrer'> مستندات </a> را بخوانید. برای افراد محتاط در زمینه امنیت لطفا <a href='https://docs.decred.org/advanced/verifying-binaries/' target='_blank' rel='noopener noreferrer'>these این مراحل </a> را برای تایید کد های باینری اپلیکیشن های دیکردتان و کیت خط فرمان دنبال کنید.",
Copy link
Member

Choose a reason for hiding this comment

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

Looks like the word these was missed?

"downloads_app_guarda": "Guarda\n با کیف پول‌های چند ارزی مبتنی بر وب، رایانه، و دستگاه همراه ارائه می‌شود.",
"downloads_app_ledger": "Ledger Live\n کیف پولی مبتنی بر وب است که با کیف پول سخت افزاری Ledger ایمن‌سازی شده است.",
"downloads_app_mobilewallets": "کیف پول های تلفن همراه دیکْرِد(Decred) موجود برای پلتفرم های اندروید و iOS",
"downloads_app_trezor": "Trezor\n یک کیف پول سخت افزاری است که توسط Decrediton and Exodus wallets پشتیبانی میشود.",
Copy link
Member

Choose a reason for hiding this comment

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

Was the word and missed?

"landing_treasury_text": "توانایی توسعه پایدار.",
"nav_become_contributor": "یک مشارکت کننده شوید",
"nav_brand_resources": "منابع نام تجاری",
"nav_bugbounty": "Bug Bounty Program",
Copy link
Member

Choose a reason for hiding this comment

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

Missed string?

"nav_brand_resources": "منابع نام تجاری",
"nav_bugbounty": "Bug Bounty Program",
"nav_community": "انجمن",
"nav_dcrdata": "Block Explorer",
Copy link
Member

Choose a reason for hiding this comment

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

Missed string?

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

2 participants