Appointment feature #9
Conversation
There was a problem hiding this comment.
Hi @Uzair-Manzoor ,
Great job so far! 👏 You've done well, and you're almost there for the next project ⏭️, but there are a few issues 🐛 that you still need to work on.
Highlights
✅ Great UI
✅ Clean Code
Required Changes ♻️
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
-[ ] Kindly ensure that add the dependencies in the PR description that you installed in the branch. It's a best practice.
| // Ensure fetchedDoctors is initialized as an array | ||
| const fetchedDoctors = useSelector((state) => state.doctors.doctors) || []; | ||
|
|
There was a problem hiding this comment.
Thank you for implementing the appointment booking feature. However, I noticed that when attempting to book an appointment, the doctor's name isn't displayed. Could you please address this issue?
| const createAppointment = createAsyncThunk('user/createAppointment', async (data) => { | ||
| try { | ||
| const response = await fetch(url, { | ||
| method: 'POST', | ||
| headers: { | ||
| 'Content-Type': 'application/json', | ||
| Authorization: `Bearer ${JSON.parse(localStorage.getItem('user'))?.token}`, | ||
| }, | ||
| body: JSON.stringify(data), |
There was a problem hiding this comment.
I encountered a 404 error when attempting to submit an appointment. It seems to be originating from the slice. Could you please investigate and resolve this issue?
There was a problem hiding this comment.
Hi @mahdinoori2000 , thanks for your feedback. I'm working to fix the addressed issues 😭
This PR has been created against below mentioned implementations into the project: