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

Translation Hooks-State.md #56

Merged
merged 5 commits into from
May 11, 2019
Merged

Translation Hooks-State.md #56

merged 5 commits into from
May 11, 2019

Conversation

J3brahimi
Copy link
Contributor

No description provided.

Copy link
Collaborator

@zahrajoulaei zahrajoulaei left a comment

Choose a reason for hiding this comment

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

سلام و ممنونم از ترجمتون
با بررسی ای که من کردم به نظر من به جز چند مورد خیلی جزیی که نوشتم متن خوب و روان بود.
بعد از اعمال تغییراتتون من اینو اکسپت میکنم و شما صبر کنید وقتی یک نفر دیگه هم اکسپت کرد مرج کنید. ممنونم

content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
@sorousht sorousht self-requested a review May 5, 2019 06:45
@sorousht
Copy link
Collaborator

sorousht commented May 5, 2019

سلام، من کمی گرفتار بودم. تا فردا شب بررسی خودم رو انجام خواهم داد.

@J3brahimi @zahrajoulaei ممنون از شما

Copy link
Collaborator

@sorousht sorousht left a comment

Choose a reason for hiding this comment

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

@J3brahimi عزیز، یکی از بهترین ترجمه‌هایی بود که تابه‌حال داشتیم. پیشنهاداتی که نوشتم بیشتر نگارشی هستند. هر موردی که نیاز به بحث داشت حتما با من در میان بگذارید.
راستی تمام کاماهای متن کاراکتر "," بود که در فارسی باید از "،" استفاده شود. این مورد در تمام متن باید اصلاح شود.
ممنونم

content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
content/docs/hooks-state.md Outdated Show resolved Hide resolved
@J3brahimi
Copy link
Contributor Author

@sorousht ممنون بابت برسی دقیقتون
تغییرات انجام شد

@sorousht
Copy link
Collaborator

sorousht commented May 8, 2019

@J3brahimi ممنون. چند مورد کوچک دیگر هم نوشتم که اگر برطرف شود تکمیل خواهد شد‌ :)

@J3brahimi
Copy link
Contributor Author

J3brahimi commented May 9, 2019

@sorousht تغییرات انجام شد

@sorousht
Copy link
Collaborator

sorousht commented May 9, 2019

@sorousht تغییرات انجام شد

در کامنت‌های قبل‌تر مواردی مطرح شده که هنوز نیاز به اصلاح دارند. در مورد اون‌ها چطور؟

@sorousht sorousht merged commit e55b002 into reactjs:fa May 11, 2019
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