reactjs / reactjs/react.dev

[Documentation] Minor change in description of state-and-lifecycle.md

Aberta
#632 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Linguagem predominante
JavaScript
Estrelas
11.8k
Forks
7.9k
Merge médio
1d 11h
PRs com merge (30d)
11

Descrição

https://github.com/reactjs/reactjs.org/blob/master/content/docs/state-and-lifecycle.md#state-updates-are-merged

The merging is shallow, so this.setState({comments}) leaves this.state.posts intact, but completely replaces this.state.comments.

Type of merging doesn't matter in this case. Neither shallow, nor deep merge can overwrite this.state.posts

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Direção de pesquisa

Abra content/docs/state-and-lifecycle.md na seção “State Updates are Merged” e revise a explicação citada sobre merging superficial. Atualize a descrição para que ela não implique que o merging superficial ou o merging profundo possam sobrescrever this.state.posts; o trabalho estará concluído quando a redação descrever o exemplo com precisão.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
javascript, react
Domínio
documentation
Tipo de issue
Documentação
Dificuldade
1/5
Tempo estimado
Menos de uma hora
Status de atividade
Estagnada
Clareza
Claramente especificada
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.