GDC Intern
Creative thinking
1x
2 Issues Created
2x
16 Issues Created
3x
128 Issues Created
4x
1024 Issues Created
Problem Solving
1x
2 Pulls Created
2x
16 Pulls Created
3x
128 Pulls Created
4x
1024 Pulls Created
Practical/Professional Skills
Communication Skills
1x
100th Community Interaction
2x
1000th Community Interaction
3x
10000th Community Interaction
Collaboration
1x
2 Pulls Created
2x
16 Pulls Created
3x
128 Pulls Created
4x
1024 Pulls Created
Community Engagement
1x
100 Activity Points
2x
1000 Activity Points
3x
10000 Activity Points
Leadership
1x
Promoted to a team leader
Learn How to Learn
1x
Completed the course work
Skills to apply digital & technology solutions
1x
1 Project Completed Successfully
2x
2 Projects Completed Successfully
3x
3 Projects Completed Successfully
Critical Thinking
Autonomy And Responsibility
1x
2 issues identified and resolved
2x
16 issues identified and resolved
3x
128 issues identified and resolved
4x
1024 issues identified and resolved
Empathy
Value inculcation
Multicultural Competence
Individual Skills
Creative thinking
Problem Solving
Practical/Professional Skills
Team Skills
Communication Skills
Collaboration
Community Engagement
Leadership
Learn How to Learn
Skills to apply digital & technology solutions
Critical Thinking
Advanced Skills
Autonomy And Responsibility
Empathy
Value inculcation
Multicultural Competence
Cultural Skills
Web Development 101
Web Development 202
Web Development 301
Teaching Assistant & Internship
Industry Placement
Alumni & Teaching Fellow
Shared a comment on a pull request in coronasafe/care_fe on November 25, 2023 at 5:41:56 PM
Also, clear the merge conflicts
https://github.com/coronasafe/care_fe/pull/6713#issuecomment-1826384897
Shared a comment on a pull request in coronasafe/care_fe on November 24, 2023 at 4:09:43 PM
@sainak
https://github.com/coronasafe/care_fe/pull/6694#issuecomment-1825866794
Shared a comment on a pull request in coronasafe/care_fe on November 24, 2023 at 7:32:59 AM
@thedevildude could you clear the merge conflicts
https://github.com/coronasafe/care_fe/pull/6596#issuecomment-1825253789
Shared a comment on an issue in coronasafe/care_fe on November 24, 2023 at 7:32:09 AM
- `Facility/PatientNotesList.tsx` - `Facility/PatientNotesSlideover.tsx` - `Patient/PatientNotes.tsx`
https://github.com/coronasafe/care_fe/issues/6705#issuecomment-1825253121
Shared a comment on an issue in coronasafe/care_fe on November 24, 2023 at 5:00:18 AM
Issue from #6484
https://github.com/coronasafe/care_fe/issues/6706#issuecomment-1825150890
Shared a comment on an issue in coronasafe/care_fe on November 23, 2023 at 6:01:39 PM
@Omkar76 you have multiple issues already assigned to you that has no PR / changes required. https://github.com/coronasafe/care_fe/issues/assigned/@me Feel free to take up once your existing assigned issues are cleared!
https://github.com/coronasafe/care_fe/issues/6705#issuecomment-1824785637
Shared a comment on a pull request in coronasafe/care_fe on November 23, 2023 at 3:21:53 PM
Could be because asset list endpoint is a redirect and fetch is no longer able to cancel it?
https://github.com/coronasafe/care_fe/pull/6695#issuecomment-1824607083
Shared a comment on a pull request in coronasafe/care_fe on November 23, 2023 at 3:20:53 PM
Just noticed, we always end up having 2 API calls for Asset List page only now :/ This at times causes behaviour to ignore the filter because the one with cached filters might respond faster than the one without cached filters. Wonder why the first query is not being aborted (although this same situation works fine by request being aborted in facility and patient list page) <img width="774" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/3b0c49b9-1554-4d2a-a421-f5f771df6128">
https://github.com/coronasafe/care_fe/pull/6695#issuecomment-1824605718
Shared a comment on an issue in coronasafe/care_fe on November 22, 2023 at 3:38:39 PM
As discussed, you can create a clone of that sheet, modify as required and keep it in the github repo itself under https://github.com/coronasafe/care_fe/tree/develop/public directory
https://github.com/coronasafe/care_fe/issues/6649#issuecomment-1823005496
Shared a comment on an issue in coronasafe/care_fe on November 22, 2023 at 3:36:44 PM
Currently, the user is able to upload results for other districts. However, the list view will only show results the user can access. So if a user from Ernakulam uploads results with the local body that belongs to Kannur, then they won't be able to preview these bulk created results in the list view as the user does not have the permission.
https://github.com/coronasafe/care_fe/issues/6650#issuecomment-1823002047
Shared a comment on an issue in coronasafe/care_fe on November 22, 2023 at 5:21:36 AM
> If I'm not mistaken, the state and district for which a user has access are stored in the state_object, and the district_object isn't, right? Correct. > Also, I have a doubt: Can a user have access to multiple states and districts? If not, we can remove these from the filtering. The highest user type we have is a State Admin, which will be linked to a particular state always. Hence the state filter seems to be a no-op. We could remove it from the filters right @nihal467? Am I missing any other cases? State Admins will be able to access multiple districts. Users less than or equal to the District Admin can have a disabled district filter with the accessible district pre-filled.
https://github.com/coronasafe/care_fe/issues/6605#issuecomment-1822132635
Shared a comment on an issue in coronasafe/care_fe on November 22, 2023 at 5:10:14 AM
Hey @krushn-a Were you able to get back at this? Are you facing any blockers?
https://github.com/coronasafe/care_fe/issues/6546#issuecomment-1822124387
Shared a comment on an issue in coronasafe/care_fe on November 22, 2023 at 3:14:20 AM
Feel free to come up with a mock-up design 😄 We can iterate on that!
https://github.com/coronasafe/care_fe/issues/6684#issuecomment-1822023033
Shared a comment on an issue in coronasafe/care_fe on November 21, 2023 at 4:08:14 PM
Ack
https://github.com/coronasafe/care_fe/issues/6686#issuecomment-1821219487
Shared a comment on a pull request in coronasafe/leaderboard-data on November 21, 2023 at 1:39:24 PM
Merging for now to get the scrapper running. Deploy will be fixed in an upcoming PR
https://github.com/coronasafe/leaderboard-data/pull/1#issuecomment-1820945370
Shared a comment on an issue in coronasafe/care_fe on November 21, 2023 at 8:04:23 AM
> Question 1 - What DevOps services and tools does your project use? For each tool, please write how it is used in the project. @sainak should be able to provide you links to the deploy docs which you can refer to. > Question 2 - What was the test coverage like? Provide "covered lines" metrics - absolute or relative (if you measured them). @adriansliva we don't have "covered lines" metrics for the front-end. However we do have one for the backend, which you can see from here. https://app.codecov.io/gh/coronasafe/care cc: @nihal467
https://github.com/coronasafe/care_fe/issues/6392#issuecomment-1820415231
Shared a comment on an issue in coronasafe/care_fe on November 21, 2023 at 6:54:22 AM
@Deepjyoti14 The PR https://github.com/coronasafe/care_fe/pull/6458 introduces support for capturing Date & Time. Once it is merged, the time part of the Admission Date & Time for existing records would be reset to 12AM (local time). I hope that'd solve this problem. cc: @aparnacoronasafe
https://github.com/coronasafe/care_fe/issues/6678#issuecomment-1820337505
Shared a comment on an issue in coronasafe/care_fe on November 21, 2023 at 3:26:49 AM
> What does it mean "only one facility is selected"? Where do I select the facility? <img width="228" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/79b95742-4042-49e3-b3da-3d2448b4c63a"> > "Make the location filter appear...", should the whole Geography section be hidden? or just the Location input? Just the location input. Currently, it's disabled, right? The same condition in which it is disabled should be used to hide/show it instead.
https://github.com/coronasafe/care_fe/issues/6671#issuecomment-1820154890
Shared a comment on an issue in coronasafe/care_fe on November 21, 2023 at 3:22:36 AM
Yes, but are you able to even read a facility that's outside of the district admin's district? If not, why bother defining that no-op check that'd always evaluate to true?
https://github.com/coronasafe/care_fe/issues/6608#issuecomment-1820152032
Shared a comment on an issue in coronasafe/care_fe on November 20, 2023 at 2:48:08 PM
@vasudevravula issue has been created. https://github.com/coronasafe/care/issues/1721
https://github.com/coronasafe/care_fe/issues/6612#issuecomment-1819206851
Shared a comment on an issue in coronasafe/care_fe on November 20, 2023 at 2:45:15 PM
Closing as, the assignees are not being marked as participants when an issue is converted to a discussion
https://github.com/coronasafe/care_fe/issues/6673#issuecomment-1819201426
Shared a comment on an issue in coronasafe/care_fe on November 20, 2023 at 6:20:04 AM
Continued in #6668 #6669
https://github.com/coronasafe/care_fe/issues/6658#issuecomment-1818294863
Shared a comment on a pull request in coronasafe/care_fe on November 20, 2023 at 2:49:41 AM
@parthksingh1 @nihal467 will be testing it this week
https://github.com/coronasafe/care_fe/pull/6661#issuecomment-1818144864
Shared a comment on an issue in coronasafe/care_fe on November 18, 2023 at 6:05:52 PM
@konavivekramakrishna The API has responded with 202 Accepted. Hence no errors were encountered. Have you tried calling the list API for the same after a successful upsert?
https://github.com/coronasafe/care_fe/issues/6650#issuecomment-1817585406
Shared a comment on an issue in coronasafe/care_fe on November 18, 2023 at 10:37:08 AM
I don't see any issue with Raviger's routing. If you look at the code, we have two routers: Session and App router. App router is accessible only if the query to get the current profile responds with 200, you'll be exposed to the App router, else it'll show the Session router. We just need to invoke the refetch of the current user query somehow whenever the JWT access token is updated in the local storage. That'd cause the AuthUserProvider component to render, and hence exposing the appropriate router based on the response. <img width="566" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/6c5c9540-0d5a-4edf-879c-6e183eb0c73a"> <img width="691" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/f180b3cb-c646-4c7e-af9c-b35e8b0af0c0">
https://github.com/coronasafe/care_fe/issues/6665#issuecomment-1817472008
Shared a comment on an issue in coronasafe/care_fe on November 17, 2023 at 4:39:36 PM
Which user can delete is already present in the code base. The issue is about hiding the button instead of keeping it disabled. <img width="474" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/dd2083b1-8996-4ecf-b883-83acdadbe4e8">
https://github.com/coronasafe/care_fe/issues/6608#issuecomment-1816743132
Shared a comment on an issue in coronasafe/care_fe on November 17, 2023 at 3:47:42 PM
This change was not reflected in front-end. Hence the issue.
https://github.com/coronasafe/care_fe/issues/6660#issuecomment-1816665898
Shared a comment on an issue in coronasafe/care_fe on November 17, 2023 at 3:47:18 PM
@ayush-seth https://github.com/coronasafe/care/commit/0cbff7c6932245cd44eaf3f4aab1ee23a9461d60 has changed the restriction to much simpler, only "StateAdmin" user can delete other users.
https://github.com/coronasafe/care_fe/issues/6660#issuecomment-1816665296
Shared a comment on an issue in coronasafe/care_fe on November 17, 2023 at 4:45:13 AM
Yup! That should be fine for now.
https://github.com/coronasafe/care_fe/issues/6612#issuecomment-1815741449
Shared a comment on a pull request in coronasafe/care_fe on November 16, 2023 at 7:19:46 AM
@sriharsh05 cypress tests are still failing
https://github.com/coronasafe/care_fe/pull/6591#issuecomment-1813908929
Shared a comment on an issue in coronasafe/care_fe on November 16, 2023 at 4:07:35 AM
We should also make the placement of the Add/Create and the Advanced Filter of Asset/Patient/User/... consistent
https://github.com/coronasafe/care_fe/issues/6648#issuecomment-1813758899
Shared a comment on a pull request in coronasafe/care_fe on November 16, 2023 at 3:32:34 AM
@sriharsh05
https://github.com/coronasafe/care_fe/pull/6591#issuecomment-1813736761
Shared a comment on an issue in coronasafe/ayushma_fe on November 16, 2023 at 2:29:22 AM
to next14?
https://github.com/coronasafe/ayushma_fe/issues/146#issuecomment-1813693180
Shared a comment on a pull request in coronasafe/care_fe on November 15, 2023 at 2:46:03 PM
See https://codeclimate.com/github/coronasafe/care_fe/pull/6645
https://github.com/coronasafe/care_fe/pull/6645#issuecomment-1812671701
Shared a comment on a pull request in coronasafe/care_fe on November 15, 2023 at 2:04:12 PM
True, but backend validations are a must anyhow. Say we create a seperate tool that populates beds (which is outside care_fe), so backend must still validate properly. But if you know for sure something is wrong and it's a minor validation, it's good to block it from front-end itself.
https://github.com/coronasafe/care_fe/pull/6643#issuecomment-1812593571
Shared a comment on an issue in coronasafe/care_fe on November 15, 2023 at 1:43:03 PM
Ref: https://github.com/coronasafe/care_fe/pull/5147
https://github.com/coronasafe/care_fe/issues/6641#issuecomment-1812559698
Shared a comment on an issue in coronasafe/care_fe on November 15, 2023 at 10:19:57 AM
@AshrafMd-1 also create a backend issue for this, to sanitize/validate the dosage fields
https://github.com/coronasafe/care_fe/issues/6625#issuecomment-1812174221
Shared a comment on an issue in coronasafe/care_fe on November 15, 2023 at 7:40:49 AM
@konavivekramakrishna this would be a backend issue
https://github.com/coronasafe/care_fe/issues/6605#issuecomment-1811950107
Shared a comment on a pull request in coronasafe/care_fe on November 14, 2023 at 11:54:33 AM
@thedevildude is this PR ready for review?
https://github.com/coronasafe/care_fe/pull/6596#issuecomment-1810064871
Shared a comment on a pull request in coronasafe/care_fe on November 14, 2023 at 8:44:29 AM
@sriharsh05 the cypress is failing
https://github.com/coronasafe/care_fe/pull/6591#issuecomment-1809764537
Shared a comment on a pull request in coronasafe/care_fe on November 14, 2023 at 8:44:00 AM
Closing as smaller PRs have been made for the same
https://github.com/coronasafe/care_fe/pull/6448#issuecomment-1809763670
Shared a comment on an issue in coronasafe/care_fe on November 14, 2023 at 7:55:16 AM
<img width="2672" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/b5022b09-70fd-4b90-9515-c78fcd8a2a0d"> <img width="2672" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/2a332284-40a0-4d1b-8955-7e09c9c41904"> cc: @aparnacoronasafe
https://github.com/coronasafe/care_fe/issues/6561#issuecomment-1809700355
Shared a comment on a pull request in coronasafe/care_fe on November 14, 2023 at 4:47:09 AM
Rebasing as lint ci is failing in installing deps.
https://github.com/coronasafe/care_fe/pull/6565#issuecomment-1809543748
Shared a comment on an issue in coronasafe/care_fe on November 10, 2023 at 4:05:42 AM
I mean, there is a back button at the top left already right? That'd be enough.
https://github.com/coronasafe/care_fe/issues/6552#issuecomment-1805054012
Shared a comment on an issue in coronasafe/care_fe on November 9, 2023 at 3:34:19 PM
Hey @krushn-a, feel free to look at the documentation https://github.com/coronasafe/care_fe/issues/6316 and the example PRs attached. Feel free to break this issue further down to multiple sub-tasks, and have multiple PR's so that it'd be easier to review and test (also to steer away from having merge conflicts). Have around 1-3 files changed at a time in a single PR.
https://github.com/coronasafe/care_fe/issues/6546#issuecomment-1804059038
Shared a comment on an issue in coronasafe/care_fe on November 9, 2023 at 3:27:01 PM
As discussed, We'll need to show a Location Select popup when opening up the camera monitoring page of a facility.
https://github.com/coronasafe/care_fe/issues/6561#issuecomment-1804045642
Shared a comment on an issue in coronasafe/care_fe on November 9, 2023 at 12:12:49 PM
The back button would suffice I guess. Let's remove both. cc: @nihal467
https://github.com/coronasafe/care_fe/issues/6552#issuecomment-1803717439
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 5:20:43 PM
@sriharsh05 the issue also mentions changes to be made to the middleware too. Are you working on it?
https://github.com/coronasafe/care_fe/pull/6479#issuecomment-1795561114
Shared a comment on an issue in coronasafe/care_fe on November 6, 2023 at 4:46:20 PM
@AshrafMd-1 you have an issue that is pending changes. Feel free to take up the issue once it's cleared. Check: https://github.com/coronasafe/care_fe/issues/assigned/@me
https://github.com/coronasafe/care_fe/issues/6550#issuecomment-1795459280
Shared a comment on an issue in coronasafe/care_fe on November 6, 2023 at 4:41:38 PM
@sriharsh05 you've an issue that is assigned to you and pending changes. Feel free to take up once it's cleared.
https://github.com/coronasafe/care_fe/issues/6550#issuecomment-1795418579
Shared a comment on a pull request in coronasafe/care on November 6, 2023 at 4:22:39 PM
@sainak need review
https://github.com/coronasafe/care/pull/1665#issuecomment-1795340899
Shared a comment on a pull request in coronasafe/care on November 6, 2023 at 4:20:55 PM
@GokulramGHV ensure your pre-commit hook is working. fix the lint errors. Lint CI is failing.
https://github.com/coronasafe/care/pull/1692#issuecomment-1795337535
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 2:43:07 PM
It's not your PR's fault
https://github.com/coronasafe/care_fe/pull/6533#issuecomment-1794986519
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 2:43:02 PM
No @konavivekramakrishna It'll fixed seperatley
https://github.com/coronasafe/care_fe/pull/6533#issuecomment-1794985864
Shared a comment on an issue in coronasafe/care_fe on November 6, 2023 at 2:04:13 PM
We could have a section with heading "Insurance Policies" and show a list of cards maybe.
https://github.com/coronasafe/care_fe/issues/6517#issuecomment-1794895277
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 1:55:16 PM
@AshrafMd-1 could you also clear the merge conflicts?
https://github.com/coronasafe/care_fe/pull/6484#issuecomment-1794878248
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 1:47:19 PM
@Ashesh3 <img width="914" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/e1dd5ceb-5e85-4f41-952e-6f520add5e02">
https://github.com/coronasafe/care_fe/pull/6379#issuecomment-1794863692
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 1:43:26 PM
<img width="1337" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/ee97111b-215d-4ffc-9df5-af126d7d66a3">
https://github.com/coronasafe/care_fe/pull/6484#issuecomment-1794856753
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 1:43:04 PM
@AshrafMd-1 fix the lint errors
https://github.com/coronasafe/care_fe/pull/6484#issuecomment-1794855909
Shared a comment on an issue in coronasafe/care on November 6, 2023 at 1:04:21 PM
Hey @satyam0601 Any updates on this? Any blockers?
https://github.com/coronasafe/care/issues/1337#issuecomment-1794773357
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 12:13:56 PM
Hey @AshrafMd-1 any updates on this?
https://github.com/coronasafe/care_fe/pull/6484#issuecomment-1794689153
Shared a comment on a pull request in coronasafe/care on November 6, 2023 at 12:08:34 PM
@cp-Coder the tests are failing
https://github.com/coronasafe/care/pull/1589#issuecomment-1794681427
Shared a comment on an issue in coronasafe/care_fe on November 6, 2023 at 7:08:12 AM
@print-Sathvik any updates on this? Any blockers?
https://github.com/coronasafe/care_fe/issues/6381#issuecomment-1794200840
Shared a comment on an issue in coronasafe/care_fe on November 6, 2023 at 6:38:13 AM
@konavivekramakrishna You've 2 issues assigned to you already that are not completed yet. https://github.com/coronasafe/care_fe/issues/assigned/@me
https://github.com/coronasafe/care_fe/issues/6546#issuecomment-1794172049
Shared a comment on an issue in coronasafe/care_fe on November 6, 2023 at 6:28:41 AM
Hey @jainvedant392 this PR is related to the PR: https://github.com/coronasafe/care_fe/pull/6528 Will assign you once it is merged.
https://github.com/coronasafe/care_fe/issues/6544#issuecomment-1794163886
Shared a comment on an issue in coronasafe/care_fe on November 6, 2023 at 6:29:04 AM
Meanwhile have a look at other issues at: https://github.com/coronasafe/care_fe/contribute
https://github.com/coronasafe/care_fe/issues/6544#issuecomment-1794164248
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 5:03:29 AM
Hey @thtauhid 👋🏼 Any updates on this?
https://github.com/coronasafe/care_fe/pull/6452#issuecomment-1794091623
Shared a comment on an issue in coronasafe/care_fe on November 6, 2023 at 4:27:40 AM
@adriansliva I've reviewed and requested some changes to the PR
https://github.com/coronasafe/care_fe/issues/6392#issuecomment-1794065955
Shared a comment on a pull request in coronasafe/care_fe on November 6, 2023 at 4:27:10 AM
@adriansliva could you also rebase and clear the merge conflicts?
https://github.com/coronasafe/care_fe/pull/6543#issuecomment-1794065658
Shared a comment on an issue in coronasafe/care_fe on November 5, 2023 at 4:04:08 PM
Yes. You can now 1. Create the branch 2. Push your changes to the branch 3. Open a PR in this repository from your branch.
https://github.com/coronasafe/care_fe/issues/6392#issuecomment-1793777261
Shared a comment on a pull request in coronasafe/care_fe on November 3, 2023 at 5:48:47 PM
@shyamprakash123 you've not solved this: https://github.com/coronasafe/care_fe/pull/6536#discussion_r1382030855
https://github.com/coronasafe/care_fe/pull/6536#issuecomment-1792881287
Shared a comment on an issue in coronasafe/care on November 3, 2023 at 9:21:06 AM
Dependent on #1690 as `PatientRelatedPermissionMixin` has been renamed to `ConsultationRelatedPermissionMixin` in it and related changes are needed to resolve this issue.
https://github.com/coronasafe/care/issues/1695#issuecomment-1792104833
Shared a comment on a pull request in coronasafe/care on November 3, 2023 at 9:14:23 AM
Additional changes: `PatientRelatedPermissionMixin` has been renamed to `ConsultationRelatedPermissionMixin` as the mixin checks upon the fields `self.patient` and `self.assigned_to` which are specific to the consultation model and not the patient model. It never failed so far in other places because: 1. It was inherited by `PatientNotes` and `Consultation` model itself. 2. It never failed for `PatientNotes` as we were not performing `edit/object read` with it (only create and list which was not dependent didn't have a check on assigned_to).
https://github.com/coronasafe/care/pull/1690#issuecomment-1792095293
Shared a comment on a pull request in coronasafe/care on October 31, 2023 at 5:18:43 AM
@Pranshu1902 marking this as do not merge as more types of diagnoses have been added in #1690
https://github.com/coronasafe/care/pull/1631#issuecomment-1786463926
Shared a comment on a pull request in coronasafe/care_fe on October 30, 2023 at 5:07:21 PM
@AshrafMd-1 any updates on this?
https://github.com/coronasafe/care_fe/pull/6484#issuecomment-1785681091
Shared a comment on an issue in coronasafe/care_fe on October 30, 2023 at 2:26:15 PM
@AshrafMd-1 this is a P1 issue, could you push it by a day or two?
https://github.com/coronasafe/care_fe/issues/6525#issuecomment-1785334186
Shared a comment on a pull request in coronasafe/care on October 29, 2023 at 4:24:17 PM
The model changes from https://github.com/coronasafe/care/issues/1574 will naturally solve the related issue.
https://github.com/coronasafe/care/pull/1673#issuecomment-1784158422
Shared a comment on a pull request in coronasafe/care_fe on October 29, 2023 at 4:22:40 PM
@GokulramGHV marking it as do not merge as the model changes made in https://github.com/coronasafe/care/issues/1574 will fix it.
https://github.com/coronasafe/care_fe/pull/6447#issuecomment-1784158006
Shared a comment on an issue in coronasafe/care_fe on October 28, 2023 at 2:21:31 PM
Access control list for Nurse and Staff: |Modules |Sub-Modules |Link |Fields |Staff (Hospital admin) |Nurse | |--------------------|--------------------|-------------------------------------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|-------------------------------------------------------------------------------|--------------------------------------------------------------| |Facility Management |Facility Details |https://care.coronasafe.in/facility/1fabc5fb-d96d-455a-ab61-53e6e0483cbc/update|All |Edit |Edit | | |Bed Capacity |https://care.coronasafe.in/facility/497fd4fc-c5c1-4ed3-9b57-64cc881087fe/bed|All |Edit |Edit | | |Doctor Capacity |https://care.coronasafe.in/facility/497fd4fc-c5c1-4ed3-9b57-64cc881087fe/doctor|All |Edit |Edit | | |Corona Triage |https://care.coronasafe.in/facility/497fd4fc-c5c1-4ed3-9b57-64cc881087fe/triage|All |Edit |Edit | | |Location/bed management|https://care.coronasafe.in/facility/497fd4fc-c5c1-4ed3-9b57-64cc881087fe/location|All |Edit |Edit | | |Delete Facility |https://care.coronasafe.in/facility/512802b9-6f09-40c6-8ff3-9cffa24bc3ad|All |NO ACCESS |NO ACCESS | |Inventory Management|All |https://care.coronasafe.in/facility/497fd4fc-c5c1-4ed3-9b57-64cc881087fe/inventory|All |Edit |Edit | |Resource Management |Raising resource request|https://care.coronasafe.in/facility/497fd4fc-c5c1-4ed3-9b57-64cc881087fe/resource/new, https://care.coronasafe.in/resource |All |Edit |Edit | |Asset Management |Registration and Updation|https://care.coronasafe.in/facility/497fd4fc-c5c1-4ed3-9b57-64cc881087fe/assets/new|All |Edit |Edit | | |asset configurations| | |no access |no access | |Sample Testing |Sample test request generation|https://care.coronasafe.in/facility/a31e6a24-0146-4043-b295-3a6c2fe7e095/patient/a709c84e-28e8-41ee-bf5e-f7beb7e859f3/sample-test|All |NO ACCESS |Edit | | |Sample test management|https://care.coronasafe.in/sample |All |NO ACCESS |NO ACCESS | |Shifting |Shifting request generation|https://care.coronasafe.in/facility/1fabc5fb-d96d-455a-ab61-53e6e0483cbc/patient/dda0caa4-a52c-49cd-9dd3-7fe8093aaa76/shift/new|All |NO ACCESS |Edit | |Patient Management |Patient Registration and updation |https://care.coronasafe.in/facility/015ad911-39b7-4104-b304-bf8178c33830/patient/76742faa-30dd-4eb9-9649-fe718f947ec0/update|All |Only view access to https://care.coronasafe.in/patients |Edit | | |Disable/Enable Transfer|https://care.coronasafe.in/facility/015ad911-39b7-4104-b304-bf8178c33830/patient/76742faa-30dd-4eb9-9649-fe718f947ec0|- |NO ACCESS |Edit | | |Consultation |https://care.coronasafe.in/facility/015ad911-39b7-4104-b304-bf8178c33830/patient/76742faa-30dd-4eb9-9649-fe718f947ec0/consultation/2ad89b7c-7bff-4035-a3c6-36d53551a8f0/update|All |NO ACCESS |Edit | | |Log Updates |https://care.coronasafe.in/facility/a31e6a24-0146-4043-b295-3a6c2fe7e095/patient/a709c84e-28e8-41ee-bf5e-f7beb7e859f3/consultation/2d6cb153-19e8-4339-8796-3c71c44cbd68/daily-rounds|All |NO ACCESS |Edit | | |Update Bed |https://care.coronasafe.in/facility/015ad911-39b7-4104-b304-bf8178c33830/patient/76742faa-30dd-4eb9-9649-fe718f947ec0/consultation/2ad89b7c-7bff-4035-a3c6-36d53551a8f0/update, https://care.coronasafe.in/facility/015ad911-39b7-4104-b304-bf8178c33830/patient/76742faa-30dd-4eb9-9649-fe718f947ec0/consultation/2ad89b7c-7bff-4035-a3c6-36d53551a8f0|Update bed in consultation form and the patient dashboard |NO ACCESS |Edit | | |Logging Investigation|https://care.coronasafe.in/facility/a31e6a24-0146-4043-b295-3a6c2fe7e095/patient/a709c84e-28e8-41ee-bf5e-f7beb7e859f3/consultation/2d6cb153-19e8-4339-8796-3c71c44cbd68/investigation/|All |NO ACCESS |Edit | | |Investigation summary generation|https://care.coronasafe.in/patient/76742faa-30dd-4eb9-9649-fe718f947ec0/investigation_reports|Generating the investigation summary to view all existing investigation reports |NO ACCESS |Edit | | |patient files (including audio recording)|https://care.coronasafe.in/facility/015ad911-39b7-4104-b304-bf8178c33830/patient/76742faa-30dd-4eb9-9649-fe718f947ec0/files/|Both file upload and voice recording |NO ACCESS |Edit | | |Discharge a patient | | |NO ACCESS |Edit | |User |Add User | | |Create users for Staff, Doctors, nurses, pharmasist, volunteer |Create users for Staff, Doctors, nurses, pharmasist, volunteer| | |Delete User | | |NO ACCESS |NO ACCESS | |Notice board |Notifying hospitals | | |NO ACCESS |NO ACCESS | | |Receiving notification through notice board| | |Yes |Yes | |Notifications |What notifications are received| | |Only Notice |All | |External Result |Uploading external result| | |NO ACCESS |Edit | | |Making patient card from external result| | |NO ACCESS |Edit |
https://github.com/coronasafe/care_fe/issues/6240#issuecomment-1783827403
Shared a comment on a pull request in coronasafe/care_fe on October 27, 2023 at 4:36:35 AM
@khavinshankar
https://github.com/coronasafe/care_fe/pull/6485#issuecomment-1782278404
Shared a comment on a pull request in coronasafe/care_fe on October 27, 2023 at 4:31:40 AM
@Ashesh3 does this seem fine?
https://github.com/coronasafe/care_fe/pull/6495#issuecomment-1782274955
Shared a comment on an issue in coronasafe/care_fe on October 26, 2023 at 3:38:21 PM
Hey @thtauhid , any updates on this?
https://github.com/coronasafe/care_fe/issues/6376#issuecomment-1781372917
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 2:48:01 PM
This design has lots of screenspace wasted IMO > @nihal467 Updated view The values are large so its best to make it full width 
https://github.com/coronasafe/care_fe/pull/6460#issuecomment-1781281740
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 2:43:42 PM
@Pranshu1902 @nihal467 These changes are likely to be already fixed in #6396 Could you test the following deploy preview? Changes in #6396 are huge and some files have been deleted/moved too. Conflicts might be chaos / practically a rework. https://deploy-preview-6396--care-net.netlify.app/facility/0c95c7f0-e1d2-4aff-83fa-933cef60d3a8/patient/7e8fc20f-02da-47f7-8c33-f2d8c7e52dd8/consultation/cb44a173-6fe2-452f-b2d1-6c6e2695411e/medicines
https://github.com/coronasafe/care_fe/pull/6460#issuecomment-1781273402
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 12:04:44 PM
You should be using the second element from the stack if it exists. (The first element would always be the current route)
https://github.com/coronasafe/care_fe/pull/6495#issuecomment-1780979256
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 12:03:01 PM
The navigate should work, why is it navigating to `/` or going into the refresh loop?
https://github.com/coronasafe/care_fe/pull/6495#issuecomment-1780976819
Shared a comment on an issue in coronasafe/care_fe on October 26, 2023 at 11:55:58 AM
@adriansliva You'll have to make the branch in your fork in order to be able to push. Refer: https://docs.github.com/en/get-started/quickstart/contributing-to-projects
https://github.com/coronasafe/care_fe/issues/6392#issuecomment-1780967139
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 11:28:01 AM
Closing as continued in #6396
https://github.com/coronasafe/care_fe/pull/6311#issuecomment-1780930466
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 11:14:20 AM
This history from useAppHistory would always match the site domain
https://github.com/coronasafe/care_fe/pull/6495#issuecomment-1780912548
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 10:45:28 AM
Rest LGTM
https://github.com/coronasafe/care_fe/pull/6461#issuecomment-1780871511
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 10:39:22 AM
The `history` from `useAppHistory` contains all previously visited URLs (as long as the page was not refreshed)
https://github.com/coronasafe/care_fe/pull/6495#issuecomment-1780862758
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 10:37:26 AM
We could use the custom hook we have `useAppHistory` (which would inherently be prone from the attack mentioned before)
https://github.com/coronasafe/care_fe/pull/6495#issuecomment-1780859929
Shared a comment on a pull request in coronasafe/care_fe on October 26, 2023 at 9:59:27 AM
@konavivekramakrishna ensure your pre-commit hook is working correctly. You can see lint issues from your files changed. <img width="1710" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/6692fce1-1b4a-4128-97b3-b617e2185264">
https://github.com/coronasafe/care_fe/pull/6461#issuecomment-1780798519
Shared a comment on an issue in coronasafe/care on October 25, 2023 at 2:33:26 PM
As discussed, a separate table is to be made, bulk_create upon consultation creation. Timestamp to be captured to align with events.
https://github.com/coronasafe/care/issues/1574#issuecomment-1779419991
Shared a comment on a pull request in coronasafe/care_fe on October 25, 2023 at 1:30:43 PM
cypress passing locally <img width="1778" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/5f3dfee0-eeba-4b03-8d28-76f69489235a">
https://github.com/coronasafe/care_fe/pull/6458#issuecomment-1779283127
Shared a comment on a pull request in coronasafe/care_fe on October 25, 2023 at 12:34:42 PM
@nihal467 backend tests fixed, cypress will require the backend to be merged to pass.
https://github.com/coronasafe/care_fe/pull/6458#issuecomment-1779176859
Shared a comment on a pull request in coronasafe/care_fe on October 25, 2023 at 10:03:30 AM
@khavinshankar
https://github.com/coronasafe/care_fe/pull/6490#issuecomment-1778927029
Shared a comment on an issue in coronasafe/care on October 25, 2023 at 9:27:39 AM
@cp-Coder unassigning as no activity for 2 weeks and as @GokulramGHV has already worked on this
https://github.com/coronasafe/care/issues/1668#issuecomment-1778868138
Shared a comment on a pull request in coronasafe/care_fe on October 25, 2023 at 7:16:30 AM
The responsive design needs to be further discussed
https://github.com/coronasafe/care_fe/pull/6396#issuecomment-1778656885
Shared a comment on an issue in coronasafe/care_fe on October 24, 2023 at 7:26:54 AM
@AshrafMd-1 any progress on this?
https://github.com/coronasafe/care_fe/issues/6372#issuecomment-1776667671
Shared a comment on a pull request in coronasafe/care_fe on October 18, 2023 at 5:00:03 AM
@nihal467 i've fixed the above mentioned issues, could you check again?
https://github.com/coronasafe/care_fe/pull/6435#issuecomment-1767644679
Shared a comment on an issue in coronasafe/care_fe on October 18, 2023 at 4:55:40 AM
Keeping the issue opened, as loadash was not removed and only replaced in previous PR
https://github.com/coronasafe/care_fe/issues/6006#issuecomment-1767640210
Shared a comment on a pull request in coronasafe/care_fe on October 18, 2023 at 3:53:49 AM
@nihal467 the other mentioned issues has been fixed. <img width="1040" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/040b7345-0650-42c0-855b-7a1145dc4a01">
https://github.com/coronasafe/care_fe/pull/6396#issuecomment-1767594846
Shared a comment on a pull request in coronasafe/care_fe on October 18, 2023 at 3:53:13 AM
The stretch issue of the page cannot be solved easily, if I make the table horizontally scrollable, it'll clip out the Popover too (all vertical content) too. This issue is a limitation by the CSS itself. https://stackoverflow.com/a/6433475/7887936 `overflow-x-auto` causes vertical contents to be clipped (that'll include the Popover overflow too) `overflow-x-auto overflow-y-visible` has no effect. cc: @Ashesh3 @khavinshankar We could make this in a dedicated page (for mobile view alone) maybe in a later PR?
https://github.com/coronasafe/care_fe/pull/6396#issuecomment-1767594503
Shared a comment on a pull request in coronasafe/care_fe on October 18, 2023 at 3:41:43 AM
@nihal467 ready for testing
https://github.com/coronasafe/care_fe/pull/6458#issuecomment-1767579074
Shared a comment on a pull request in coronasafe/care_fe on October 18, 2023 at 3:39:13 AM
@nihal467 nope, that's not an intentional behaviour as per discussions made. My bad, I had accidentally forgot to remove the range constraints.
https://github.com/coronasafe/care_fe/pull/6458#issuecomment-1767577332
Shared a comment on a pull request in coronasafe/care_fe on October 17, 2023 at 2:49:43 PM
As per discussion, the administration comment should be shown along with the administration event itself instead of a seperate event. And it should be greyed out too.
https://github.com/coronasafe/care_fe/pull/6311#issuecomment-1766578047
Shared a comment on an issue in coronasafe/care_fe on October 17, 2023 at 12:54:14 PM
@cp-Coder any updates on the front-end PR? This issue will be unassigned from you due to no activity tonight.
https://github.com/coronasafe/care_fe/issues/6240#issuecomment-1766361741
Shared a comment on a pull request in coronasafe/care_fe on October 12, 2023 at 9:17:53 AM
You could do: ```jsx <span class="capitalize">{"COMPLETED".toLowerCase()}</span> // 👉 Completed ```
https://github.com/coronasafe/care_fe/pull/6415#issuecomment-1759240041
Shared a comment on an issue in coronasafe/care_fe on October 11, 2023 at 6:36:27 AM
@nihal467 is this issue still relevant? Users can always narrow down the search using the filters (created date, origin/approving/assigned facility. If not shall we move this to wishlist?
https://github.com/coronasafe/care_fe/issues/5695#issuecomment-1756923150
Shared a comment on an issue in coronasafe/care_fe on October 11, 2023 at 4:36:18 AM
Hey @AlexVCS, we'll be unassigning you from this issue to keep it open for others as the related PR has become stale. Feel free to ping if you are facing any blockers or checkout other good first issues.
https://github.com/coronasafe/care_fe/issues/6328#issuecomment-1756775731
Shared a comment on a pull request in coronasafe/care_deploy_configs on October 9, 2023 at 4:18:23 PM
@mathew-alex summary of this change: `jwt_token_refresh_interval` config can be set to change the JWT token refresh interval. This is not a mandatory config, since the front-end has a fallback defined.
https://github.com/coronasafe/care_deploy_configs/pull/18#issuecomment-1753305189
Shared a comment on a pull request in coronasafe/care_fe on October 9, 2023 at 5:53:04 AM
@kshitijv256 https://github.com/coronasafe/care_deploy_configs/pull/18 is not required to be merged for this to work right, since we have a fallback default specified in the front end. @khavinshankar this should be good to be merged.
https://github.com/coronasafe/care_fe/pull/6418#issuecomment-1752382705
Shared a comment on an issue in coronasafe/care_fe on October 6, 2023 at 2:37:22 PM
As discussed: - Brought Dead can be merged with Outpatient since all such records' Decision after consultation is already set to "Declare Death", so no information is lost even if merged. - Transfer from ICU and Ward can also be merged. The Name of ICU/Ward would be a Facility Location select dropdown, and later on we can allow a location to be classified as ICU/Ward instead.
https://github.com/coronasafe/care_fe/issues/6280#issuecomment-1750791148
Shared a comment on an issue in coronasafe/care_fe on October 6, 2023 at 1:19:09 PM
Also, should the Name of the Ward/ICU field be a select dropdown that the user can select from the list of locations available in that facility? Or should it simply be a text field?
https://github.com/coronasafe/care_fe/issues/6280#issuecomment-1750670360
Shared a comment on an issue in coronasafe/care_fe on October 6, 2023 at 12:56:02 PM
something like this: <img width="405" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/cd3c543a-6adc-481a-bb19-12e67124ba10"> <img width="416" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/f46de325-de85-41ea-aa5d-787f569c0788">
https://github.com/coronasafe/care_fe/issues/6280#issuecomment-1750632801
Shared a comment on an issue in coronasafe/care_fe on October 6, 2023 at 12:47:21 PM
Same suggestion for the Internal Transfer within the Facility (ICU/Ward) We could do like: if the user selects "Internal transfer within facility", another dropdown can be shown to select ward or ICU. Merging options would mean we would lose the level of detail from the existing data too.
https://github.com/coronasafe/care_fe/issues/6280#issuecomment-1750611171
Shared a comment on an issue in coronasafe/care_fe on October 6, 2023 at 12:41:43 PM
@aparnacoronasafe If we merge "brought dead" with "outpatient", we'll no longer be able to know which all patients were brought dead (even for existing consultations). A better alternative would be to Show "Outpatient/Emergency Room" itself in the dropdown, however, if the user selects this option, a checkbox would appear whether the patient was "Brought Dead" or not. For other options, this brought dead checkbox would stay hidden. This way we won't lose the clarity on our existing data. What are your thoughts?
https://github.com/coronasafe/care_fe/issues/6280#issuecomment-1750598310
Shared a comment on an issue in coronasafe/care_fe on October 6, 2023 at 5:53:52 AM
Both live and all export is working. Able to modify registered date too without issues. <img width="1707" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/87c70091-7db1-431c-b619-a34d5f4e2cd9">
https://github.com/coronasafe/care_fe/issues/6406#issuecomment-1750022914
Shared a comment on an issue in coronasafe/care_fe on October 6, 2023 at 5:52:28 AM
@nihal467 unable to reproduce both issues mentioned
https://github.com/coronasafe/care_fe/issues/6406#issuecomment-1750021868
Shared a comment on an issue in coronasafe/care on October 5, 2023 at 6:49:47 PM
duplicate of #1658
https://github.com/coronasafe/care/issues/1659#issuecomment-1749462931
Shared a comment on an issue in coronasafe/care_fe on October 4, 2023 at 1:20:52 PM
All results with: `entityType > 0` has this mentioned issue.
https://github.com/coronasafe/care_fe/issues/5293#issuecomment-1746867295
Shared a comment on an issue in coronasafe/care_fe on October 4, 2023 at 1:20:08 PM
we may not be able to use `theCode` for lookup. Attaching an example entity with `"theCode": "PE95/"` and `"title": "Assault by exposure to or harmful effects of other inorganic substances"`. This entity is not present in our `icd11.json` nor the spreadsheet export from ICD11's browsing tool. A similar one exists with `PE95`, title: `"Assault by exposure to or harmful effects of other or unspecified substances chiefly nonmedicinal as to source"`. <img width="1710" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/8af60147-25d2-4335-951c-01ac0c2d4a35">
https://github.com/coronasafe/care_fe/issues/5293#issuecomment-1746865818
Shared a comment on a pull request in coronasafe/care_fe on October 4, 2023 at 12:54:02 PM
@khavinshankar
https://github.com/coronasafe/care_fe/pull/6338#issuecomment-1746820788
Shared a comment on a pull request in coronasafe/care on October 4, 2023 at 12:37:06 PM
marking as hold for https://github.com/coronasafe/care_fe/issues/5293
https://github.com/coronasafe/care/pull/1636#issuecomment-1746791189
Shared a comment on an issue in coronasafe/care_fe on October 3, 2023 at 2:09:39 PM
As per discussions, we need to bin every 4 hours to 1 column
https://github.com/coronasafe/care_fe/issues/6341#issuecomment-1745059629
Shared a comment on an issue in coronasafe/care_fe on October 3, 2023 at 7:18:23 AM
Decided to only freeze the columns
https://github.com/coronasafe/care_fe/issues/6177#issuecomment-1744346268
Shared a comment on an issue in coronasafe/care_fe on October 3, 2023 at 5:47:22 AM
We've decided to use the ICD API docker; FE would hit the ICD API for searching; and use the `theCode` attribute to perform a lookup on to care's ICD entries upon form submission.
https://github.com/coronasafe/care_fe/issues/5293#issuecomment-1744251600
Shared a comment on a pull request in coronasafe/care_fe on October 3, 2023 at 3:12:02 AM
@kshitijv256 check the cypress tests failing
https://github.com/coronasafe/care_fe/pull/6374#issuecomment-1744096007
Shared a comment on a pull request in coronasafe/care_fe on October 1, 2023 at 9:24:08 AM
Nice work @jevil25 , rest everything LGTM
https://github.com/coronasafe/care_fe/pull/6333#issuecomment-1742019607
Shared a comment on a pull request in coronasafe/care_fe on September 30, 2023 at 12:46:05 PM
yes @shyamprakash123
https://github.com/coronasafe/care_fe/pull/6344#issuecomment-1741758534
Shared a comment on a pull request in coronasafe/care_fe on September 29, 2023 at 11:39:34 AM
@kshitijv256 when making changes, feel free to split the entire refactor into smaller PRs if that would help. It would be easier to make the changes, QA testing, and review.
https://github.com/coronasafe/care_fe/pull/6370#issuecomment-1740751784
Shared a comment on an issue in coronasafe/care_fe on September 29, 2023 at 7:45:46 AM
@aparnacoronasafe @SeyiesilieChadi we have decided to add an edit button, that behind the scenes will discontinue and create a new prescription with the edited attributes. @aparnacoronasafe just wanted to confirm, whether should we restrict the editing of certain fields (in both normal and PRN cases) - Medicine will not be editable - Normal prescription CANNOT be converted to PRN prescription and vice versa via this method. Should any other fields be restricted from edit? <img width="1822" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/b3a4b9be-ba5e-4961-866c-e2fa69017add"> <img width="1822" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/cb1cb73d-aff6-4f29-88fb-6f565ff28b6f">
https://github.com/coronasafe/care_fe/issues/6340#issuecomment-1740457221
Shared a comment on a pull request in coronasafe/care_fe on September 28, 2023 at 9:15:21 AM
That can be a separate issue
https://github.com/coronasafe/care_fe/pull/6345#issuecomment-1738772821
Shared a comment on a pull request in coronasafe/care_fe on September 26, 2023 at 1:29:26 PM
Refer: https://github.com/coronasafe/care_fe/issues/4467 @suprabathk
https://github.com/coronasafe/care_fe/pull/6338#issuecomment-1735547125
Shared a comment on a pull request in coronasafe/care_fe on September 26, 2023 at 1:00:45 PM
Understood, if that's the case then isn't it a backend issue since the backend is giving the wrong value for `last_administered_on`? We are returning `created_date` instead of `administered_date` in backend. <img width="686" alt="image" src="https://github.com/coronasafe/care_fe/assets/25143503/360d746f-28cd-4d39-9e27-8be3f22f7bd4">
https://github.com/coronasafe/care_fe/pull/6345#issuecomment-1735498078
Shared a comment on an issue in coronasafe/care_fe on September 26, 2023 at 7:18:44 AM
@SeyiesilieChadi @aparnacoronasafe Need more clarity on the reasons for the edit. Whether it's a valid change as patients condition changes (whether these changes to be tracked or not), or is it a correction due to wrong data entry, or any other reason?
https://github.com/coronasafe/care_fe/issues/6340#issuecomment-1734968354
Shared a comment on a pull request in coronasafe/care_fe on September 26, 2023 at 1:59:07 AM
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.
https://github.com/coronasafe/care_fe/pull/6345#issuecomment-1734713016
Shared a comment on an issue in coronasafe/care_fe on September 25, 2023 at 3:30:37 PM
@SeyiesilieChadi @aparnacoronasafe just to clarify, this is to support editing of "prescription" or "administrations"?
https://github.com/coronasafe/care_fe/issues/6340#issuecomment-1733959332
Shared a comment on an issue in coronasafe/care_fe on September 25, 2023 at 11:24:12 AM
Yes. This was intentionally designed to not support edits. To change the dosage/frequency or any other attributes, the user will have to discontinue the ongoing prescription, provide a reason for discontinuation (whether it was an accidental error in the entry or an intentional change), and then create a new one.
https://github.com/coronasafe/care_fe/issues/6340#issuecomment-1733473312
Shared a comment on an issue in coronasafe/care on September 25, 2023 at 6:07:06 AM
the field name can be taken for renaming along with the migrations of events module
https://github.com/coronasafe/care/issues/1618#issuecomment-1732978855
Shared a comment on a pull request in coronasafe/care_fe on September 25, 2023 at 4:20:37 AM
Closed in favour of #6296
https://github.com/coronasafe/care_fe/pull/6221#issuecomment-1732882829
Shared a comment on a pull request in coronasafe/care_fe on September 23, 2023 at 10:35:27 AM
You can add the response types to the API route def. itself in the `api.tsx` file itself as mentioned in the EPIC issue linked to the issue. You can add another key value pair, example: ``` getPatient: { path: "/api/v1/....", method: "GET", TRes: Res<PatientModel>(), } ``` If PatientModel or (any other similar name) doesn't exist that defines the type for that API response, you'll have to create one. Check the README of this repo. and you'll get to see the swagger documentation. You could use tools like https://transform.tools/json-to-typescript to convert the example response to JSON. (Related models needs to be properly referenced to other interfaces however)
https://github.com/coronasafe/care_fe/pull/6329#issuecomment-1732277792
Shared a comment on an issue in coronasafe/care_fe on September 23, 2023 at 3:07:16 AM
The latest date should always be the latest of all last administered date. i.e. Medicine A is last administered 23rd 10:02 AM, Medicine B is last administered 23rd 11:04 AM, then it should show the column upto 23rd 11:00 AM (because that column contains all administrations between 11 AM and 11:59 AM)
https://github.com/coronasafe/care_fe/issues/6323#issuecomment-1732193065
Shared a comment on a pull request in coronasafe/care_fe on September 22, 2023 at 1:53:37 PM
Ideally, you'll need to run cypress tests against ur local backend with the default dummy data
https://github.com/coronasafe/care_fe/pull/6318#issuecomment-1731462727
Shared a comment on an issue in coronasafe/care_fe on September 21, 2023 at 8:28:47 AM
Remaining migrations tracked in #6316
https://github.com/coronasafe/care_fe/issues/6058#issuecomment-1729099694
Shared a comment on an issue in coronasafe/care_fe on September 21, 2023 at 8:27:08 AM
Will be covered via #6316
https://github.com/coronasafe/care_fe/issues/6150#issuecomment-1729097170