-
Notifications
You must be signed in to change notification settings - Fork 260
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
Translate DOM Elements #120
Conversation
Deploy preview for idreactjs ready! Built with commit c5e3dd3 |
Deploy preview for id-reactjs ready! Built with commit c5e3dd3 |
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.
Ada beberapa isu typo dan penerjemahan yang perlu dibetulkan sebelum di-merge ya.
thanks for the feedback guys @resir014 @paramithatm, i have done the changes |
thanks feedbacknya @vferdiansyah, sudah saya update ya. |
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.
After these new changes, LGTM!
shouldn't this be merged already @resir014? |
@vferdiansyah Sure, I'll take a quick look first. |
Mohon reviewnya semua, sedikit catatan: saya kurang yakin apakah style dan children perlu diterjemahkan ? di PR ini tidak saya terjemahkan, hanya diberi italic.