dwyl/learn-redux

'addTodo' and 'setVisibilityFilter' actions are unused in index.html example

Open

#50 aperta il 11 set 2017

Vedi su GitHub
 (0 commenti) (1 reazione) (0 assegnatari)HTML (42 fork)batch import
help wanted

Metriche repository

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

Descrizione

I was just annotating a print out of the code to remind myself of some of the concepts learned in the video tutorials, and found the following issues:

  • addTodo action is unused: should replace the dispatch action in the AddTodo component
  • setVisibilityFilter action is unused: should be used by the dispatch action in the mapDispatchToLinkProps method

Guida contributor

Direzione di ricerca
Il problema identifica che le azioni 'addTodo' e 'setVisibilityFilter' sono dichiarate ma non utilizzate nell'esempio index.html. La correzione prevede di sostituire le chiamate dispatch dirette nel componente AddTodo e in mapDispatchToLinkProps con questi creatori di azioni. Non esistono PR o commenti collegati, quindi la soluzione richiede la modifica del file index.html per importare e utilizzare correttamente le azioni.
Tech stack
javascriptreacthtml
Dominio
frontenddocumentation
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
basic Redux conceptsfamiliarity with React components
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85