Skip to content

translate updating arrays page to persian #467

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

translate updating arrays page to persian #467

wants to merge 7 commits into from

Conversation

KianoshArian
Copy link

This pull request includes the translation of the updating-arrays-in-state page.
Please let me know if you have any suggestions for improvement.
Thanks.

@vercel Vercel
Copy link

vercel bot commented May 15, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fa-react-dev 🔄 Building (Inspect) Visit Preview 💬 Add feedback Jun 8, 2025 2:47pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
fa-legacy-reactjs-org ⬜️ Ignored (Inspect) Visit Preview Jun 8, 2025 2:47pm

@KianoshArian KianoshArian mentioned this pull request May 21, 2025
Closed
mrbadri
mrbadri reviewed Jun 7, 2025
View reviewed changes
Copy link
Collaborator

@mrbadri mrbadri left a comment

Choose a reason for hiding this comment

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

سلام @KianoshArian عزیز!
از همکاری و وقتی که برای ترجمه مستندات ری‌اکت فارسی گذاشتی، ممنون هستیم.
در ادامه، تغییرات پیشنهادی رو ارسال میکنم.

همچنین برای اطمینان بیشتر از اینکه همه‌ی اصلاحات کامل انجام شده باشه، می‌تونید از گزینه‌ی Commit suggestion استفاده کنید.

روش استفاده از این Action رو هم براتون ضمیمه کردم

image

@KianoshArian @mrbadri
Apply suggestions from code review
fdfb9d6 
Apply the suggested edits to translation

Co-authored-by: Mohammad Reza Badri <[email protected]>
@KianoshArian @mrbadri
Apply suggestions from code review
7a1bba8 
Co-authored-by: Mohammad Reza Badri <[email protected]>
@KianoshArian
Update updating-arrays-in-state.md
296b9f6 
fix right and left translation mistake
@KianoshArian
Copy link
Author

KianoshArian commented Jun 8, 2025 •
edited
Loading

Dear @mrbadri,
I reviewed and applied all of your suggestions, making a few changes that I noted in the comments. Please let me know if anything else needs to be addressed.

@KianoshArian
Copy link
Author

Dear @mrbadri
In case you missed my reply in the review section, I have already committed all the suggestions in this PR, including the fix for rtl translation suggested by myself. Therefore all the previous changes are complete now.

Thank you for your time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mrbadri mrbadri mrbadri left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@KianoshArian @mrbadri