supasate/connected-react-router

Browser history stack is modified when time travelling

Open

#36 aberto em 19 de fev. de 2017

Ver no GitHub
 (0 comments) (3 reactions) (0 assignees)JavaScript (606 forks)batch import
bughelp wanted

Métricas do repositório

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

Description

Currently, when we do time travelling, the ConnectedRouter tries to change the URL bar to match the router state in the redux store (which is changed by Redux DevTools) by using history.push (see here).

This method seems to work fine, however, internally, the real browser history stack is modified (new history is pushed on top of the stack). It would be better if we can change the URL by just moving the pointer in the history stack to match time travelling.

Guia do colaborador

Direção de pesquisa
Substitua history.push por history.replace no ConnectedRouter.js para evitar modificar a pilha do histórico do navegador ao viajar no tempo. Alternativamente, use history.go para ajustar o ponteiro da pilha para o estado correto sem adicionar novas entradas.
Pilha de tecnologia
javascriptreact
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-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptReactRedux
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60