streamich/react-use

Remove warnings from `useDeepCompareEffect`

Open

#755 aberto em 6 de nov. de 2019

Ver no GitHub
 (8 comments) (0 reactions) (0 assignees)TypeScript (3.273 forks)batch import
buggood first issuehelp wanted

Métricas do repositório

Stars
 (43.979 stars)
Métricas de merge de PR
 (Mesclagem média 1743d 9h) (1 fundiu PR em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Encontre a implementação de useDeepCompareEffect no código fonte (provavelmente em src/useDeepCompareEffect.ts). Examine a lógica de aviso e decida se deve removê la ou adicionar uma opção de supressão. Leia os comentários da issue para orientação. Certifique se de que quaisquer alterações não quebrem os testes existentes.
Pilha de tecnologia
typescriptreact
Domain
frontend
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
React hooks
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70