-
Notifications
You must be signed in to change notification settings - Fork 493
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
Replaced useDispatch w. useQuery/request: Users (src/Components/Users/**) [5 out of 5 components] #6596
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@thedevildude is this PR ready for review? |
No @rithviknishad it needs few more commits. Should by ready by tomorrow's EOD |
@thedevildude #6652 solve this issue as well along with this PR |
@nihal467 Yeah sure, I'll work on this too |
LGTM |
👋 Hi, @thedevildude, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
@thedevildude fix the merge conflicts |
Co-authored-by: Rithvik Nishad <[email protected]>
WHAT
🤖 Generated by Copilot at 7a7ad19
Refactored the user-related components and API routes to use custom hooks and models instead of redux actions and dispatches. Added
GenderType
andweekly_working_hours
to user models inmodels.tsx
to enhance user profile data. Improved type safety and readability of API calls inapi.tsx
by addingTRes
andmethod
properties.Proposed Changes
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
HOW
🤖 Generated by Copilot at 7a7ad19
TRes
property to define response types for api routes inapi.tsx
(link,link,link,link,link)method
property to define request methods for api routes inapi.tsx
(link,link)GenderType
type andweekly_working_hours
property to user models inmodels.tsx
(link,link,link,link)