Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix useEffect mount/unmount usage #852

Merged
merged 1 commit into from
May 16, 2019
Merged

Conversation

jigsawye
Copy link
Contributor

The second params of useEffect should be [], otherwise it will be called when re-render every time.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.7%) to 91.845% when pulling 90a7e5c on jigsawye:patch-1 into 8d1e5d4 on i18next:master.

@jamuhl
Copy link
Member

jamuhl commented May 16, 2019

You're right...will be merged and published asap. Thanks for catching that one...

@jamuhl jamuhl merged commit e26547f into i18next:master May 16, 2019
@jigsawye jigsawye deleted the patch-1 branch May 16, 2019 13:33
@jamuhl
Copy link
Member

jamuhl commented May 16, 2019

was published in react-i18next@10.9.1

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants