appsmithorg/appsmith

(JS-0458) Definitions for unused states detected

Open

#7 943 ouverte le 29 sept. 2021

Voir sur GitHub
 (20 commentaires) (0 réactions) (1 assigné)TypeScript (4 508 forks)batch import
Good First IssueLow effortTask

Métriques du dépôt

Stars
 (39 454 stars)
Métriques de merge PR
 (Merge moyen 6j 14h) (26 PRs mergées en 30 j)

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/

Guide contributeur

Direction de recherche
Examinez les 9 occurrences listées dans le lien DeepSource (deepsource.io/gh/appsmithorg/appsmith/issue/JS 0458/occurrences/). Chaque occurrence pointe vers un fichier et un numéro de ligne où une définition d'état inutilisée est détectée. Pour chaque cas, vérifiez que la variable d'état n'est effectivement utilisée nulle part dans le composant ou ses enfants. Si elle est inutilisée, supprimez la définition d'état et tous les appels de setter associés. Assurez vous que le composant fonctionne toujours correctement en vérifiant toute utilisation indirecte. Le dépôt est basé sur React avec TypeScript, recherchez donc les appels useState dans les composants fonction. Le problème a un assigné, coordonnez si nécessaire.
Stack technique
typescriptreact
Domaine
frontend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptReact
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60