> @print-Sathvik any updates on this? Any blockers? Yes, I am unable to add insurance details. /payors is giving 501 bad gateway error I am taking Khavin's help

https://github.com/coronasafe/care_fe/issues/6381#issuecomment-1794220673

I had to take leave unexpectedly. I will continue from today.

https://github.com/coronasafe/care_fe/issues/6381#issuecomment-1767575418

I am closing this PR as the issue is fixed by this commit: b7fc53c670bb85e68631f8ce3b02defcda675ae7

https://github.com/coronasafe/care_fe/pull/6345#issuecomment-1744951248

> At 8:50 AM if I administer a medicine and set the administration time as 4:30 AM then currently we can see columns upto 9-10 slot as latest_administered_on is 8:50 AM. @rithviknishad I fixed it as you suggested by modifying in useRangePagination hook but "last modified" part on the top of the table is shown as 4:30 AM as per the above example. But it should be shown as 8:50AM. There is another attribute `modified_date` in the response of the endpoint `/api/v1/consultation/{consultation_external_id}/prescriptions/` which I think should actually contain the last time the prescription is updated which is 8:50 AM as per above example but the `modified_date` contains the same value as `created_date`. I think this should also be fixed on the backend only. Previously before the above fix on backend we used `last_administered_on` for this part.

https://github.com/coronasafe/care_fe/pull/6345#issuecomment-1738769865

I will remove that N additional queries part now and also the additional state as we can use the last_administered_on due to above fix on backend

https://github.com/coronasafe/care_fe/pull/6345#issuecomment-1735601737

> You need not create additional state either to store last administered. The problem is within how the slots are generated by the useRangePagination hook alone. So the issue is to remove only 1 additional column that is ahead of the last_administered_on time. At 8:50 AM if I administer a medicine and set the administration time as 4:30 AM then currently we can see columns upto 9-10 slot as latest_administered_on is 8:50 AM. I changed this to show only upto slot 4-5 But what I need to do is to display slots upto 8-9 only right?

https://github.com/coronasafe/care_fe/pull/6345#issuecomment-1735330628

> The prescription record itself has last_administered_on field that will give u the latest date without fetching administrations. The last_administered_on is storing the time at which the administration was made by the user instead of storing the latest administered time given by the user. If a medicine is administered right now(current time: 8:50 AM) to a previous time, let's say to 4:30 AM then 5-8 slots will also be displayed even though latest administration was at. 4:30 AM

https://github.com/coronasafe/care_fe/pull/6345#issuecomment-1734767194

> End date: 23/09/2023 Sorry for taking an extra day. Before assigning this issue to myself I got the solution and thought that I will be completing it in few minutes but I misunderstood the issue earlier before your explanation. After trying out a different approach I noticed that the 1st administration is not being displayed in the table when the 1st administration was sent into second page of paginated table. I thought it was because of my work and I was making changes to my work but that bug is already existing at care.ohc.network. I fixed that too.

https://github.com/coronasafe/care_fe/issues/6323#issuecomment-1734220722

End date: 23/09/2023

https://github.com/coronasafe/care_fe/issues/6323#issuecomment-1732192367

> Just a minor suggestion, you could've moved the phone number related utility methods to a seperate file under say: `src/Utils/PhoneNumberUtils.ts` > > Currently, the `utils.ts` file feels too big and we should consider splitting it based on relevance. I pulled, rebased and separated phone number utils in my local but that requires me to force push to current branch. Doing that will close this PR. So I will create a new PR for seperating phone number utils once this PR is merged.

https://github.com/coronasafe/care_fe/pull/6222#issuecomment-1722923046

@rithviknishad New changes: - Phone number in excel file for importing asset does not have country code. So I adjusted that part by adding country code before validating. - After uploading the excel file location with value "Camera Locations" was being selected but there is no such location, so I changed it to "Camera Loc" as this is used in other tests.

https://github.com/coronasafe/care_fe/pull/6222#issuecomment-1715575901

I am closing this PR as the diff shows others' work as part of this PR

https://github.com/coronasafe/care_fe/pull/6205#issuecomment-1704408266

Hey @mustafaazad03, sorry for creating PR on the same issue. The last time I checked this at night, there was no one assigned to this. So I started working and forgot to assign it to myself. Guess we started almost at the same time. I checked your latest PR, setting the limit to null removes it from the URL but that breaks the purpose of limit. There are 14 patients displayed per page instead of 12. Also the pagination component which is used to navigate to different set of patients is replaced by red colored text saying "Do not render Pagination component, when limit is <= 0" I changed my PR to a draft PR. So let me know if you can resolve this, I will close my PR.

https://github.com/coronasafe/care_fe/issues/6101#issuecomment-1699925957

@nihal467 There is some work going on on redesigning doctors notes by C4GT interns. So I stopped working on this issue's outcomes might clash with their design

https://github.com/coronasafe/care_fe/issues/4705#issuecomment-1662141147

Ok, will do that tonight

https://github.com/coronasafe/care_fe/pull/5720#issuecomment-1614406077

Can someone else take this up. When I fixed this some other issues starte. > @print-Sathvik @rithviknishad  > > the map is not properly aligned, make it appear near the icon

https://github.com/coronasafe/care_fe/pull/5719#issuecomment-1614102822

Replacing MateriaUI is completed but the GLocationPicker element is not working properly in mobile view and at lower screen width, even in the live deployment. When the location select button is clicked, the search bar and close button are occupying the entire popover element without displaying the map. @rithviknishad Should I create a PR for the completed part and create new issue for GLocationPicker?

https://github.com/coronasafe/care_fe/issues/4952#issuecomment-1595744824

Will create a PR now in 30 mins

https://github.com/coronasafe/care_fe/issues/4977#issuecomment-1595696247

@sachdevavaibhav checkout the epic issue 5006. There you will find many such MatrialUI sub tasks

https://github.com/coronasafe/care_fe/issues/4941#issuecomment-1574994619

Hey @sachdevavaibhav , I have completed it. I will be creating a PR evening.

https://github.com/coronasafe/care_fe/issues/4941#issuecomment-1574909992

Hey @sachdevavaibhav , I am done with it. I will be creating a PR evening.

https://github.com/coronasafe/care_fe/issues/4941#issuecomment-1574909991

Yes @rithviknishad , I changed the branch now. Also fixed the same issue with Location Filter and created a new PR(5574) as this cannot be reopened.

https://github.com/coronasafe/care_fe/pull/5559#issuecomment-1570511292

By mistake, I made some other changes on my current branch. So I added padding and created a new PR (5563). So I am closing this PR now. From next time I will continue committing to the same branch for additional changes

https://github.com/coronasafe/care_fe/pull/5542#issuecomment-1567139814