-
Notifications
You must be signed in to change notification settings - Fork 496
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 with useQuerry on DailyRoundListDetails #6696
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. |
👋 Hi, @krushn-a, This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there. |
you should be doing: const { res, data, loading } = useQuery(...); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Data
is not being used anywhere, right?- What exactly does
fetchpatient
do? I don't see it doing any data fetching.
@krushn-a Kindly have a look at the example PR's referenced in the useQuery EPIC issue. |
Hi, This pr has been automatically marked as stale because it has not had any recent activity. It will be automatically closed if no further activity occurs for 7 more days. Thank you for your contributions. |
I have gone through the refrences and put all my effort to make appropiate changes, i hope it will pass. |
I am continuing to make changes in other files also |
Hi, This pr has been automatically marked as stale because it has not had any recent activity. It will be automatically closed if no further activity occurs for 7 more days. Thank you for your contributions. |
@krushn-a what is the status on this PR cc: @rithviknishad |
Hi, @gigincg, @nihal467, @khavinshankar, @mathew-alex, This pr has been automatically closed because it has not had any recent activity. Thank you for your contributions. Feel free to repopen the pr. |
WHAT
copilot:
Proposed Changes
useDispatch
w.useQuery
/request
: Patient (src/Components/Patient/**
) #6546@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖[deprecated] Generated by Copilot at b3fea0a
dispatch
andgetConsultationDailyRoundsDetails
actions with custom hookuseConsultationQuery
that usesuseQuery
androutes.getDailyReport
to fetch daily round details inDailyRoundListDetails.tsx
(link, link, link, link)?.
) to accesscurrent_health
andmedication_given
properties ofres.data
object inDailyRoundListDetails.tsx
to prevent errors when they are null or undefined (link, link)DailyRoundsModel
type frommodels
module and use it to define response type ofroutes.getDailyReport
object inapi.tsx
(link, link)method
property toroutes.getDailyReport
object inapi.tsx
to specify GET request (link)