acdlite/redux-router

Router state duplication

Open

#59 aperta il 18 set 2015

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (215 fork)batch import
help wanted

Metriche repository

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

Descrizione

It seems to me that there is some overhead with keeping router state. When using redux-react-router we have router state in our app state and also in the component's props. So we have

this.props.router.location this.props.router.component this.props.router.params this.props.router.routes

this.props.params this.props.location this.props.history this.props.routes this.props.route this.props.routeParams

Is there any way to reduce number of props kept?

Guida contributor

Direzione di ricerca
Esamina il codice sorgente di redux router per capire come le props vengono iniettate sia dallo store Redux che dal componente, e identifica quali props possono essere consolidate o rimosse per ridurre la duplicazione.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Refactoring
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaScriptReactReduxReact Router
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50