Skip to content

Conversation

@palanikalyan
Copy link

The codebase had a TODO recommending typed hooks for Redux (see store.ts). Using untyped useDispatch() risks losing type information and causes boilerplate (casting). Providing centralized typed hooks (useAppDispatch/useAppSelector) improves developer ergonomics, reduces type errors, and makes future frontend contributions safer and easier to review.
The change is small and low-risk, making it a good first contribution: it removes a TODO and modernizes a part of the frontend without changing runtime behavior.

@vercel
Copy link

vercel bot commented Oct 18, 2025

@palanikalyan is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Collaborator

@ManishMadan2882 ManishMadan2882 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a compilation error, running npm i && npm run dev in frontend dir

@ManishMadan2882
Copy link
Collaborator

Closing this PR due to inactivity, feel free to re-raise in case the issue requested changes are made
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants