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

Remoção da lógica de encontrar o TODO pelo ID passado por parametro #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

felipealmeidal3
Copy link

Estavamos passando o ID do TODO na função onToggle, quando poderiamos passar o TODO e modificar o estado dele mesmo, já que é mesmo da lista 'this.state.todoList' (OBS: a mudança de estado da lista é um problema?)

… passar o TODO e modificar o estado dele mesmo, já que é mesmo da lista 'this.state.todoList' (OBS: a mudança de estado da lista é um problema?)
@guiportooo
Copy link

guiportooo commented Jun 6, 2017

@felipealmeidal3 Acho que funciona sim, mas dá uma olhada na seção State Updates May Be Asynchronous desse link: https://facebook.github.io/react/docs/state-and-lifecycle.html
Pelo que eu entendi, pelo fato dos updates serem assincronos não é garantido que o objeto todo estará no estado que nós esperamos no momento que cair na função onToggle.
Mas acho que esse comportamento também pode afetar a maneira como está hoje, com o map.

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.

None yet

2 participants