Contributor
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
Still waiting for this
care/1399
added tests for `Bed`, `AssetBed` and `ConsultationBed`care/1370
prefetched skills for userassigned serializer in patientconsultation ā¦care/1344
implementing manual logic of on_delete option in M2M models care/1329
Delete incomplete file uploadscare_fe/5550
Added None option in admitted_to bed filters so that we can filter for patient who are admitted but no bed assignedcare/1318
Added a way to handle filtering admitted but no bed assigned Shared a comment on an issue in coronasafe/care on June 19, 2023 at 4:09:39 PM
Hey @cp-Coder , how about u create issues for the next 4 , from assetbed to consultationbed and link them in EPIC issue. So that I can work on them. š
https://github.com/coronasafe/care/issues/1385#issuecomment-1597443232
Shared a comment on a pull request in coronasafe/care on June 14, 2023 at 7:57:02 PM
@vigneshhari @sainak @rithviknishad Added tests , Please Review!!
https://github.com/coronasafe/care/pull/1318#issuecomment-1591888745
Shared a comment on an issue in coronasafe/care on June 12, 2023 at 5:44:14 PM
Creating Tests for the whole code base is itself a huge task. So, Lemme know if I can make some part of contributions to this issue.
https://github.com/coronasafe/care/issues/1367#issuecomment-1587789387
Shared a comment on a pull request in coronasafe/care on June 12, 2023 at 5:40:35 PM
@vigneshhari This is a PR, I 've created Earlier. Closed the old one. I added the tests . Please Review it!
https://github.com/coronasafe/care/pull/1370#issuecomment-1587781049
Shared a comment on a pull request in coronasafe/care on June 9, 2023 at 9:50:42 PM
> You could have gotten away without mocking in these tests by using the update method of the django queryset ( they do not invoke the save methods and the created timestamps can be added manually "to my knowledge" ), even in this case, we are not mocking the database interactions, we are just mocking the time method. In almost all cases we would not want to mock the database, the database is what we are usually testing against. mocking is useful when there is an external service involved, like an SMS API or an integration with an API, in our unit tests we mock all the responses they can give ( success, failure, exception ) and test if the function we wrote handles every case correctly. Yaa ik that i haven't mocked any database interactions in the pushed code . I just asked it coz I first wrote the test by mocking database interactions, later i realised that isn't a gud idea. So i changed it. Thanks for sharing ur opinion. :smile:
https://github.com/coronasafe/care/pull/1328#issuecomment-1585163250
Shared a comment on a pull request in coronasafe/care on June 9, 2023 at 9:40:36 PM
> @mathew-alex are S3 objects versioned in any deployment ? Hey I have gone through the code base quite a few times, wherever s3 configs or s3 methods made use of. No where Versioning details are mentioned. if versioning exists, it has to dealt while creation and retrieving of objects. So, ig none of our existing deployments are using versioned s3 buckets.
https://github.com/coronasafe/care/pull/1329#issuecomment-1585153772
Shared a comment on a pull request in coronasafe/care on June 7, 2023 at 3:18:29 PM
@sainak Please review this
https://github.com/coronasafe/care/pull/1344#issuecomment-1581043770
Shared a comment on a pull request in coronasafe/care on June 7, 2023 at 2:30:59 PM
@sainak Review please
https://github.com/coronasafe/care/pull/1329#issuecomment-1580949272
Shared a comment on a pull request in coronasafe/care on June 3, 2023 at 10:30:33 AM
> @yaswanthsaivendra also could you prefix the issue numbers with "Fixes" or other linking keywords? My bad, I did all changes in single commit š®āšØ. Referenced the changes in each task.
https://github.com/coronasafe/care/pull/1344#issuecomment-1574846047
Shared a comment on a pull request in coronasafe/care on June 3, 2023 at 10:28:17 AM
> You are performing `<QuerySet>.delete()`. However, we've overridden the `<Instance>.delete()` to soft delete only. Would `<QuerySet>.delete()` call the overridden `delete` method and soft delete or does it hard delete? š¤¦š hard delete, made changes to soft delete
https://github.com/coronasafe/care/pull/1344#issuecomment-1574845725
Shared a comment on a pull request in coronasafe/care on June 2, 2023 at 11:00:01 PM
@rithviknishad Review please!
https://github.com/coronasafe/care/pull/1344#issuecomment-1574414851
Shared a comment on a pull request in coronasafe/care on June 2, 2023 at 9:10:06 PM
Updated the code - to make bulk delete from s3, thereby reducing the number of api requests.
https://github.com/coronasafe/care/pull/1329#issuecomment-1574319921
Shared a comment on a pull request in coronasafe/care on June 2, 2023 at 7:18:57 AM
@rithviknishad @vigneshhari review pls
https://github.com/coronasafe/care/pull/1329#issuecomment-1573272503
Shared a comment on a pull request in coronasafe/care on June 1, 2023 at 12:58:21 PM
> @yaswanthsaivendra there are no changes in this pr you should create a new branch for every pull request sorry, I messed a bit. Pushed the changes Now. The worst thing, I did at the start is making changes in my master branch which leads to all this mess in the commits. Will avoid it in my further prs.
https://github.com/coronasafe/care/pull/1310#issuecomment-1572007187
Shared a comment on an issue in coronasafe/care on May 31, 2023 at 10:30:08 AM
Hey @rithviknishad , May ik what kind of query profiling you wanted to achieve. Ig , we already sql query calculation with the help of `django_debug_toolbar` rgt. Isn't that sufficient??
https://github.com/coronasafe/care/issues/1321#issuecomment-1569932968
Shared a comment on an issue in coronasafe/care on May 31, 2023 at 10:27:44 AM
> i want to resolve this issue, can i start ? No this one is still need to be discussed. Meanwhile , you can work on other issues.
https://github.com/coronasafe/care/issues/1326#issuecomment-1569929154
Shared a comment on a pull request in coronasafe/care on May 30, 2023 at 12:34:24 PM
@rithviknishad , @vigneshhari Please review!!
https://github.com/coronasafe/care/pull/1328#issuecomment-1568356094
Shared a comment on an issue in coronasafe/care on May 29, 2023 at 9:10:49 PM
Hello team, Lemme know if its a relevant issue, if so I would like to work upon it!
https://github.com/coronasafe/care/issues/1326#issuecomment-1567520778
Shared a comment on an issue in coronasafe/care on May 29, 2023 at 7:48:06 PM
Hey , i would like to take up this one. As far i understand, we would need to create a periodic_task using celery that would run maybe every midnight to delete all the notifications that are older than 90 days. but can we specify a few things for me, - Like what would be a great place to create this scheduler function(maybe inside a new file - `care/facility/tasks/notification/delete_older_notifications.py`)?? - period/Time to run the task(maybe every midnight)?? - I cant see a `created_at` field so do we need to decide this 90 days older on basis upon `read_at` field which exists in Notification table or add a `created_at` field to the table??
https://github.com/coronasafe/care/issues/1320#issuecomment-1567462513
Shared a comment on an issue in coronasafe/care on May 28, 2023 at 4:49:25 PM
Is this issue still relevant, can I take up this one?? And as far I understand 2nd part of it, It requires to create a seperate through table ConsultationBedAsset with the schema of something like this, ConsultationBedAsset: consultation_bed - ForeignKey asset - ForeignKey but i didn't get the purpose of BedFlag through table . can u explain?
https://github.com/coronasafe/care/issues/1103#issuecomment-1566197659
Shared a comment on a pull request in coronasafe/care on May 27, 2023 at 1:41:11 PM
Hey @sainak , I made the necessary changes , can u review it please!
https://github.com/coronasafe/care/pull/1310#issuecomment-1565424239
Shared a comment on a pull request in coronasafe/care on May 26, 2023 at 8:56:08 PM
@sainak UserAssingedSerializer is used at 2 places. 1. in FacilityUserViewSet, where I have made the changes to prefetch the skills. 2. in [PatientConsulationSerializer](https://github.com/coronasafe/care/blob/7d14e41251787fe1bbb952ac22278f998b86b24a/care/facility/api/serializers/patient_consultation.py#L68) here i am not sure how it can be done. Can u suggest , what can be done here?
https://github.com/coronasafe/care/pull/1310#issuecomment-1564941244
Shared a comment on an issue in coronasafe/care on May 24, 2023 at 10:36:35 AM
@rithviknishad I have created a PR (#1310 ) for this issue. please review it!
https://github.com/coronasafe/care/issues/1189#issuecomment-1560871888