nfl/react-helmet

onChangeClientState not executing

Open

#328 aberto em 23 de out. de 2017

Ver no GitHub
 (2 comments) (8 reactions) (0 assignees)JavaScript (706 forks)batch import
bughelp wanted

Métricas do repositório

Stars
 (17.167 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

It appears that after v5.0.0 onChangeClientState has some odd behaviour - its seemingly overridden by nested <Helmet /> usage or something else that results in it not executing when there are is more than one use of it. This wasn't the case before and would executed on the component it was specified for as well as all others.

This is useful for running code after Helmet has injected all the specified items - componentDidMount will sometimes return null when querying the DOM for items that should have been injected so we cannot use this.

Guia do colaborador

Direção de pesquisa
Comece examinando o código fonte do react helmet, focando no método componentWillMount e na implementação de onChangeClientState. Veja o log do git para as alterações entre v4 e v5 para identificar o que mudou. Leia os dois comentários na issue para obter dicas adicionais. Considere escrever uma reprodução mínima no CodeSandbox para isolar o bug.
Pilha de tecnologia
reactjavascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Precisa de investigação
Prerequisites
ReactJavaScriptcomponent lifecycle
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
15