appsmithorg/appsmith

(JS-0458) Definitions for unused states detected

Aberta

#7.943 aberto em 29 de set. de 2021

Ver no GitHubGuia do colaborador
 (21 comentários) (0 reação) (1 responsável)TypeScript (4.508 forks)batch import
Good First IssueLow effortTask

Métricas do repositório

Stars
 (39.454 estrelas)
Métricas de merge de PR
 (Mesclagem média 6d 14h) (26 fundiu PRs em 30d)

Description

Description

You should avoid defining states that are not being used because unused states might affect while refactoring the codebase and introduces unnecessary operations by ReactJS. It is recommended to plan the behavior and operation of the component.

Occurrences

There are 9 occurrences of this issue in the repository.

See all occurrences on DeepSource → deepsource.io/gh/appsmithorg/appsmith/issue/JS-0458/occurrences/

Guia do colaborador

Direção de pesquisa
Revise as 9 ocorrências listadas no link do DeepSource (deepsource.io/gh/appsmithorg/appsmith/issue/JS 0458/occurrences/). Cada ocorrência aponta para um arquivo e número de linha onde uma definição de estado não utilizada é detectada. Para cada caso, verifique se a variável de estado realmente não é usada em nenhum lugar do componente ou de seus filhos. Se não for usada, remova a definição de estado e quaisquer chamadas de setter associadas. Certifique se de que o componente ainda funcione corretamente verificando se há algum uso indireto. O repositório é baseado em React com TypeScript, portanto, procure por chamadas useState em componentes de função. A issue tem um responsável atribuído, então coordene se necessário.
Pilha de tecnologia
typescriptreact
Domain
frontend
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
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
JavaScriptReact
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60