Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(auth): optimize fetchUser with useCallback and remove unnece… #285

Merged
merged 1 commit into from
Mar 20, 2025

Conversation

24Victor
Copy link
Contributor

This pull request includes changes to the resources/ts/hooks/useAuth.ts and resources/ts/layouts/AdminLayout.tsx files to improve code efficiency and readability. The most important changes include the addition of useCallback for memoization, and the removal of an unnecessary state variable.

Improvements to useAuth hook:

Code cleanup in AdminLayout:

@24Victor 24Victor requested a review from a team as a code owner March 19, 2025 20:01
@github-actions github-actions bot added kind/bug Categorizes issue or PR as related to a bug. area/javascript Categorizes issue or PR as related to JavaScript. labels Mar 19, 2025
@24Victor 24Victor merged commit 22f4303 into main Mar 20, 2025
5 checks passed
@24Victor 24Victor deleted the fix/fetchUser branch March 20, 2025 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/javascript Categorizes issue or PR as related to JavaScript. kind/bug Categorizes issue or PR as related to a bug.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants