Skip to content

Conversation

@BChoudhury-ms
Copy link
Collaborator

Preview this branch

Description

There was the below comment to remove useMemo & useCallback to remove the over optimization
#2241 (comment)

I think useCallback here is using more resources than it's saving. You should probably only use useCallback (or useMemo) if either A) you're making a call over the wire and want to cache the result for future renders or B) using it noticeably improves performance of the page (which is almost never the case).
See https://kentcdodds.com/blog/usememo-and-usecallback

@BChoudhury-ms BChoudhury-ms requested a review from a team as a code owner November 13, 2025 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants