quran/quran.com-frontend-next

Standardize using dispatch

Aperta

#642 aperta il 31 ott 2021

Vedi su GitHubGuida contributor
 (8 commenti) (0 reazioni) (0 assegnatari)TypeScript (616 fork)auto 404
help wanted

Metriche repository

Star
 (1888 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

In our codebase we are having 2 ways to call dispatch:

  1. dispatch(decreaseQuranTextFontScale())
  2. dispatch({ type: toggleIsOpen.type });

We should follow one of them for consistency.

Guida contributor

Direzione di ricerca
Trova tutte le chiamate dispatch nel codice. Determina quale pattern (action creator o oggetto) è più comune o preferito. Quindi aggiorna tutte le occorrenze per utilizzare quel pattern.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptRedux
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90