-
Notifications
You must be signed in to change notification settings - Fork 103
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
[Done] translate "How to Contribute" #120
Conversation
Deploy preview for ar-reactjs ready! Built with commit a46aab7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @Fcmam5 , thank you so much for the great job and your effort you're doing 🎓.
Please, I suggested some request to change some expression 📑. Can you check it out✒️?!
You may be right but this is just a view to be a clearer expression ✌🏻✔️.
@3imed-jaberi this PR is in "progress", I think that it doesn't make sense to review an unfinished work since it will have more commits for other parts of the page, and also some corrections when re-reading the translation |
I'm sorry .. I just want to give you some help in your first commit .. |
Hi @3imed-jaberi, please review this PR after it's been completed. and merge it when approved. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
شكرا لك اخي الكريم على عملك الرائع و مجهود في مساهماتك القيمة 💪🏻 ..
لقد قمت بإقتراح بعض التعديلات بالإضافة إلى التعديلات السابقة ..
يمكنك النظر إليها و شكرا لك
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉Amazing job
No description provided.