dwyl/learn-redux

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

Offen

#50 geöffnet am 11.09.2017

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (1 Reaktion) (0 zugewiesene Personen)HTML (42 Forks)batch import
help wanted

Repository-Metriken

Stars
 (446 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Überprüfen Sie die Datei index.html im Repository. Finden Sie Stellen, an denen die Aktionen addTodo und setVisibilityFilter definiert, aber nicht verwendet werden. Ersetzen Sie die dispatch Aufrufe in der AddTodo Komponente und in mapDispatchToLinkProps durch die entsprechenden Aktionsersteller.
Tech Stack
javascriptreact
Domain
frontend
Issue Type
Dokumentation
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
1
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85