supasate/connected-react-router

Browser history stack is modified when time travelling

Open

#36 aperta il 19 feb 2017

Vedi su GitHub
 (0 commenti) (3 reazioni) (0 assegnatari)JavaScript (606 fork)batch import
bughelp wanted

Metriche repository

Star
 (4741 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Sostituisci history.push con history.replace in ConnectedRouter.js per evitare di modificare lo stack della cronologia del browser durante il viaggio nel tempo. In alternativa, usa history.go per regolare il puntatore dello stack allo stato corretto senza aggiungere nuove voci.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptReactRedux
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60