Skip to content
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

Wrong translation in docs/hooks-state.html #163

Closed
1more opened this issue Aug 9, 2019 · 1 comment · Fixed by #164
Closed

Wrong translation in docs/hooks-state.html #163

1more opened this issue Aug 9, 2019 · 1 comment · Fixed by #164

Comments

@1more
Copy link
Contributor

1more commented Aug 9, 2019

Hi,
I am a beginner of react-hooks and I really appreciates for your translation works.

I found a small mistranslation in https://reactjs.org/docs/hooks-state.html#declaring-a-state-variable page.

  • English version
What do we pass to useState as an argument? The only argument to the useState() Hook is the initial state. 
Unlike with classes, the state doesn’t have to be an object.
...
  • Korean version
useState의 인자로 무엇을 넘겨주어야 할까요? useState()Hook의 인자로 넘겨주는 값은 state의 초기 값입니다.
함수 컴포넌트의 state는 클래스와 달리 객체를 가질 수 없고, 숫자 타입과 문자 타입을 가질 수 있습니다. 

the state doesn't have to be an object is translated as 객체를 가질 수 없고 for now,
but 객체일 필요가 없고 could be more correct one for this case.

Thanks.

@taehwanno
Copy link
Member

Thanks for reporting this. the suggestion is fine but I prefer to 객체일 필요는 없고.
can you fix this and make some pull request?

taehwanno pushed a commit that referenced this issue Aug 26, 2019
* Fix mistranslation in a hooks-state doc

* Clean up textlint disable comment
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 a pull request may close this issue.

2 participants