-
Notifications
You must be signed in to change notification settings - Fork 507
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: Shifting (src/Components/Shifting/**) #6533
Replaced useDispatch w. useQuery/request: Shifting (src/Components/Shifting/**) #6533
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. |
8afac1e
to
ffcb37d
Compare
f89be30
to
be4a00c
Compare
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.
Nice work @konavivekramakrishna LGTM
@rithviknishad, |
No @konavivekramakrishna |
It's not your PR's fault |
Got it👍 |
https://deploy-preview-6533--care-net.netlify.app/shifting/c8bcd2a4-ed41-4b64-ba27-96cd62a978dc this no comment is not in staging, so just remove it pagination for the comments is breaking |
f822986
to
3c4f224
Compare
LGTM |
❌ Deploy Preview for care-net failed.
|
WHAT
🤖 Generated by Copilot at a3582c4
The pull request refactors the shifting features of the frontend application to use the
useQuery
hook and therequest
function from theUtils
folder, which simplify and improve the API requests and responses logic. The pull request also defines new interfaces for the shift and shift details data, and updates the existing interfaces and routes for the shifting features. The pull request also removes some unused and redundant code, and adds some user interface enhancements.Proposed Changes
useDispatch
w.useQuery
/request
: Shifting (src/Components/Shifting/**
) #6483@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at a3582c4
state
property optional in theFacilityModel
interface and theexclude_user
prop optional in theFacilitySelectProps
interface (link, link)useQuery
andrequest
functions from theUtils
folder, which use theroutes
object and theaxios
library to make and handle API requests (link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link, link)routes
object, using theIShift
andIShiftDetails
interfaces from themodels
file and theType
function from theruntypes
library (link, link, link)routes
object, using theUserModel
interface and thePaginatedResponse
type (link)Comment
insrc/Components/Resource/CommentSection.tsx
, which takes anIComment
object as a prop and renders the comment details (link)CommentSection
component insrc/Components/Shifting/CommentsSection.tsx
with thePaginatedList
component from theCAREUI
folder, which handles the loading and pagination logic internally (link)src/Components/Shifting/ShiftDetails.tsx
andsrc/Components/Shifting/ListView.tsx
, which show messages based on the shift status or confirmation (link, link)src/Components/Shifting/ShiftDetails.tsx
, which allow the user to make a call or navigate to the patient details page by clicking on them (link, link)offset
state variable insrc/Components/Shifting/ShiftingBoard.tsx
, which is used to keep track of the pagination offset for the API request (link)