antonioru/beautiful-react-hooks

useDebouncedCallback doesn't consider its dependencies

Ouverte

#420 ouverte le 29 nov. 2023

Voir sur GitHubGuide contributeur
 (3 commentaires) (2 réactions) (0 personne assignée)JavaScript (581 forks)batch import
buggood first issuehelp wanted

Métriques du dépôt

Stars
 (8 350 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

Describe the bug

This is the same issue reported in #386 for useDebouncedCallback, I don't know why that issue has been closed. I'd expect that any dependency given as args will invalidate the callback but that's not the case because deps are not given to the useEffect call here. As result the callback sees the values as one step behind.

To Reproduce

Here's a codesandbox to reproduce the issue https://codesandbox.io/p/sandbox/pedantic-grass-f9f6qw

Expected behavior

When I click on useDebouncedCallback button I should see same values as other buttons in console, but I see the old counter value.

Screenshots

image

Guide contributeur

Direction de recherche
Ajoutez le tableau de dépendances à l'appel useEffect dans useDebouncedCallback pour garantir que le callback soit mis à jour lorsque les dépendances changent.
Stack technique
typescriptreact
Domaine
frontend
Type d'issue
Bug
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.
Moins d'une heure
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
ReactJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70