streamich/react-use

Remove warnings from `useDeepCompareEffect`

Aperta

#755 aperta il 6 nov 2019

Vedi su GitHubGuida contributor
 (8 commenti) (0 reazioni) (0 assegnatari)TypeScript (3273 fork)batch import
buggood first issuehelp wanted

Metriche repository

Star
 (43.979 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Is your feature request related to a problem? Please describe.

useDeepCompareEffect gives warnings when using it with primitive dependencies. This is frustrating for the common case where dependencies could either undefined or an object that you'd like to be deep-compared.

Describe the solution you'd like

Remove warnings from useDeepCompareEffect or provide a way to suppress warnings.

Guida contributor

Direzione di ricerca
Trova l'implementazione di useDeepCompareEffect nel codice sorgente (probabilmente in src/useDeepCompareEffect.ts). Esamina la logica di avviso e decidi se rimuoverla o aggiungere un'opzione di soppressione. Leggi i commenti dell'issue per indicazioni. Assicurati che eventuali modifiche non interrompano i test esistenti.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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
React hooks
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70