🐛 Fix Bug In useDebouncedCallback Doesn't Consider Its Dependencies (#420)

[baradoner]

🐛 Fix Bug In useDebouncedCallback Doesn't Consider Its Dependencies (#420)

Description

This commit addresses an issue in the useDebouncedCallback hook where the dependencies were not properly taken into account. This bug caused the callback function to not update correctly when its dependencies changed. The fix ensures that the callback function is debounced and correctly re-evaluates its dependencies, improving the hook's reliability and performance.

Related Issue

useDebouncedCallback doesn't consider its dependencies #420

Motivation and Context

This change is required to ensure that the useDebouncedCallback hook functions as expected by properly considering its dependencies. Without this fix, users may encounter unexpected behavior where the callback does not update in response to changes in its dependencies, leading to potential bugs and inconsistencies in the application.

How Has This Been Tested?

Unit tests verified the callback re-evaluates its dependencies. All tests passed successfully.

Screenshots (if appropriate):

[antonioru]

Amazing work, thank you!

@baradoner can you please update the version in the package.json from 5.0.2 to 5.0.3 so that it will automatically trigger a new release?

[baradoner]

Done. ‫בתאריך יום ו׳, 30 באוג׳ 2024 ב-17:26 מאת ‪Rù‬‏ ***@***.*** ‬‏>:‬

…

Amazing work, thank you! @baradoner <https://github.com/baradoner> can you please update the version in the package.json from 5.0.2 to 5.0.3 so that it will automatically trigger a new release? — Reply to this email directly, view it on GitHub <#426 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BCKGP5MGJF642PXIBJASMBLZUB6IJAVCNFSM6AAAAABLAT57YWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRRGQZTCOJRGM> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[moooss]

As useThrottleCallback works exactly the same way, wouldn't be the right place to do the same change over there also ?

[baradoner]

I see. I'll fix it there too.

…

On Thu, Nov 28, 2024, 18:38 Mathieu Mousnier ***@***.***> wrote: As useThrottleCallback works exactly the same way, wouldn't be the right place to do the same change over there also ? — Reply to this email directly, view it on GitHub <#426 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BCKGP5J36DVFFF7EQ7IPTTL2C5BIXAVCNFSM6AAAAABLAT57YWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBWGQ4DQMBQG4> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[baradoner]

Done. ‫בתאריך יום ו׳, 29 בנוב׳ 2024 ב-9:33 מאת ‪Avraham Bar Adon‬‏ <‪ ***@***.***‬‏>:‬

…

I see. I'll fix it there too. On Thu, Nov 28, 2024, 18:38 Mathieu Mousnier ***@***.***> wrote: > As useThrottleCallback works exactly the same way, wouldn't be the right > place to do the same change over there also ? > > — > Reply to this email directly, view it on GitHub > <#426 (comment)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/BCKGP5J36DVFFF7EQ7IPTTL2C5BIXAVCNFSM6AAAAABLAT57YWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBWGQ4DQMBQG4> > . > You are receiving this because you were mentioned.Message ID: > ***@***.***> >