Skip to content

Translate Introducing JSX #7

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 8 commits into from
Feb 11, 2019
Merged

Translate Introducing JSX #7

merged 8 commits into from
Feb 11, 2019

Conversation

phonglk
Copy link
Contributor

@phonglk phonglk commented Feb 10, 2019

  • introduction
  • why-jsx
  • embedding-expressions-in-jsx
  • jsx-is-an-expression-too
  • specifying-attributes-with-jsx
  • specifying-children-with-jsx
  • jsx-prevents-injection-attacks
  • jsx-represents-objects

@phonglk phonglk changed the title trans why jsx [WIP] Translate Introducing JSX Feb 10, 2019
@phonglk phonglk changed the title [WIP] Translate Introducing JSX Translate Introducing JSX Feb 10, 2019
Copy link
Collaborator

@nampdn nampdn left a comment

Choose a reason for hiding this comment

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

Chân thành cảm ơn bạn @phonglk, mình đã xem qua và có gợi ý bạn một vài chỉnh sửa. Có gì nhờ bạn điều chỉnh thêm một chút rồi chị @ltmylinh review lần nữa sẽ merge nhé!

@nampdn nampdn requested a review from ltmylinh February 10, 2019 14:56
Copy link
Collaborator

@ltmylinh ltmylinh left a comment

Choose a reason for hiding this comment

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

Great translation!!! I learned some words from you @phonglk.

@tesseralis tesseralis mentioned this pull request Feb 11, 2019
95 tasks
@phonglk
Copy link
Contributor Author

phonglk commented Feb 11, 2019

Cám ơn anh chị đã góp ý, em đã address tất cả những comment trên. Em sẽ rút kn cho �document kế. Anh chị xem lại lần nữa giúp em.

@nampdn nampdn merged commit b2a9136 into reactjs:master Feb 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.

3 participants