Skip to content

Translate Don't Call PropTypes Warning #174

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

Merged
merged 4 commits into from
Sep 15, 2021
Merged

Translate Don't Call PropTypes Warning #174

merged 4 commits into from
Sep 15, 2021

Conversation

kctrnn
Copy link
Contributor

@kctrnn kctrnn commented Sep 15, 2021

No description provided.

@netlify
Copy link

netlify bot commented Sep 15, 2021

✔️ Deploy Preview for vi-reactjs-org ready!

🔨 Explore the source changes: b1e4677

🔍 Inspect the deploy log: https://app.netlify.com/sites/vi-reactjs-org/deploys/61417b4b8bb219000713013d

😎 Browse the preview: https://deploy-preview-174--vi-reactjs-org.netlify.app

@Ryan-LT
Copy link
Member

Ryan-LT commented Sep 15, 2021

Bạn ơi, Bản dịch khá tốt. Nhưng bạn có thể check lại một vòng giúp mình được không? Vài chỗ mình đọc thấy vẫn hơi lấn cấn, có vài từ technical thì ko cần dịch sát nghĩa sang tiêng Việt đâu.

Mình cảm ơn bạn!

@kctrnn
Copy link
Contributor Author

kctrnn commented Sep 15, 2021

Hi anh, em có chỉnh sửa lại đôi chút. Đây là lần đầu tiên em đóng góp vào một thứ như này và trình độ của em cũng còn yếu. Mong nhận được góp ý từ anh 🤗

};
}
```

This will silence the warning.
Điều này sẽ tắt tiếng cảnh báo.
Copy link
Member

Choose a reason for hiding this comment

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

Warning này là log warning ở console nha em, em thay đổi lại giúp a chỗ này nhé. Khúc này nếu đổi lại sẽ phù hợp hơn nè

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Điều này sẽ tắt log warning ở console

Như này được không anh?

Copy link
Member

Choose a reason for hiding this comment

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

"Điều này sẽ giúp xoá warning ở console"

Như vầy được ko em nè?

Copy link
Member

Choose a reason for hiding this comment

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

Làm mất cảnh báo thôi em, "ngưng" hoặc "tắt" nghĩa là tắt luôn rồi, Anh sợ sẽ dễ hiểu lầm là tắt luôn log

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Dạ em vừa update ạ. Cảm ơn anh nhiều hehe

@Ryan-LT
Copy link
Member

Ryan-LT commented Sep 15, 2021

Hi anh, em có chỉnh sửa lại đôi chút. Đây là lần đầu tiên em đóng góp vào một thứ như này và trình độ của em cũng còn yếu. Mong nhận được góp ý từ anh 🤗

Ok. Ko sao nha e. Em dịch rồi duyệt lại dưới phương diện người mới, học đọc hiểu tốt là được thôi nè. Nếu còn lấn cấn thì cứ làm từ từ thôi em. Cái này ko có deadline đâu :D

@Ryan-LT Ryan-LT merged commit 600768c into reactjs:main Sep 15, 2021
@Ryan-LT Ryan-LT mentioned this pull request Sep 15, 2021
95 tasks
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.

2 participants