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
care_fe/6845
use celery beat to check for pending migrationscare/1755
remove prescription suppliers componentscare_fe/6457
add support for webrtc streamcare/1579
refactor all files as per ruff rulescare/1559
feat: use ruffmedibase/7
Update dependenciescare/1349
use signals to update patient and bed count of facilitiesShared a comment on an issue in coronasafe/care_fe on December 12, 2023 at 12:37:53 PM
Some notes don't have a consultation linked to them So we should display all notes for a patient I. The patient details page cc @Ashesh3 @rithviknishad
https://github.com/coronasafe/care_fe/issues/6818#issuecomment-1851956587
Shared a comment on a pull request in coronasafe/care on December 11, 2023 at 4:52:06 PM
@dependabot rebase
https://github.com/coronasafe/care/pull/1694#issuecomment-1850473719
Shared a comment on an issue in coronasafe/care_fe on December 10, 2023 at 4:49:57 PM
@jainvedant392 I would suggest setting up the backend instance locally and modifying any existing route to throw an error, that should help you test different cases of 5xx being raised by the backend but, the structure of the error response would be the same as 4xx errors.
https://github.com/coronasafe/care_fe/issues/6815#issuecomment-1849017984
Shared a comment on an issue in coronasafe/care_fe on December 10, 2023 at 4:40:27 PM
hey @jainvedant392, yes this issue is open for contributors
https://github.com/coronasafe/care_fe/issues/6815#issuecomment-1849015699
Shared a comment on a pull request in coronasafe/care on December 10, 2023 at 11:38:47 AM
As we checked more middleware instances, we discovered that this only happening with https://dev_middleware.coronasafe.live/
https://github.com/coronasafe/care/pull/1759#issuecomment-1848939915
Shared a comment on a pull request in coronasafe/care on December 8, 2023 at 7:49:28 AM
@dependabot rebase
https://github.com/coronasafe/care/pull/1694#issuecomment-1846714097
Shared a comment on a pull request in coronasafe/care on December 7, 2023 at 8:13:49 PM
@dependabot rebase
https://github.com/coronasafe/care/pull/1686#issuecomment-1846042725
Shared a comment on a pull request in coronasafe/care on December 7, 2023 at 6:26:07 PM
@vigneshhari bumping it due to security issues; [no breaking changes introduced](https://github.com/python-pillow/Pillow/blob/main/CHANGES.rst)
https://github.com/coronasafe/care/pull/1677#issuecomment-1845890176
Shared a comment on a pull request in coronasafe/care on December 7, 2023 at 6:14:54 PM
@dependabot rebase
https://github.com/coronasafe/care/pull/1677#issuecomment-1845874482
Shared a comment on a pull request in coronasafe/care on December 7, 2023 at 6:14:18 PM
@dependabot reopen
https://github.com/coronasafe/care/pull/1677#issuecomment-1845873614
Shared a comment on a pull request in coronasafe/care on December 7, 2023 at 6:13:18 PM
@dependabot recreate
https://github.com/coronasafe/care/pull/1677#issuecomment-1845872330
Shared a comment on a pull request in coronasafe/care on December 7, 2023 at 6:05:50 PM
@dependabot rebase
https://github.com/coronasafe/care/pull/1694#issuecomment-1845861642
Shared a comment on a pull request in coronasafe/care on December 7, 2023 at 5:56:31 PM
> [#1704 (files)](https://github.com/coronasafe/care/pull/1704/files#diff-855ccb735aeda2b9a06ef57530782964236126df0b0ff032ddb451c6e9046945R65) AuthZ is already present The change is for adding for write authz
https://github.com/coronasafe/care/pull/1704#issuecomment-1845846994
Shared a comment on a pull request in coronasafe/care on December 7, 2023 at 10:49:32 AM
@GokulramGHV when you are using TextChoices you don't need to access the value of the enum states, so you can use `PrescriptionDosageType.REGULAR` as a parameter directly
https://github.com/coronasafe/care/pull/1692#issuecomment-1845117225
Shared a comment on a pull request in coronasafe/care on December 6, 2023 at 11:54:35 AM
@nihal467 @GokulramGHV can you verify if this is not affecting other district fields?
https://github.com/coronasafe/care/pull/1735#issuecomment-1842724699
Shared a comment on a pull request in coronasafe/care on December 6, 2023 at 9:50:54 AM
<img width="309" alt="image" src="https://github.com/coronasafe/care/assets/46787056/086befba-35bb-48d0-ad83-f386784147a8"> 😅 i thought its deleted
https://github.com/coronasafe/care/pull/1748#issuecomment-1842540968
Shared a comment on a pull request in coronasafe/care on December 6, 2023 at 9:48:45 AM
> Hey, I am not deleting any files 😅 Just added more facilities and patients to the existing dummy data. I think the change size `+1 −3,128` might have been confusing since the new data is all in a single line. This is to discourage someone manually working with the json file directly. Oh so that's why the title 😅, formatting the json gave more clarity, can you regenerate the fixtures by excluding, `facility.historicalpatientregistration` and `facility.historicalfacilitycapacity` they are not required and half of the file size and the `facility_users.json` seems redundant, so both the command and file can be removed
https://github.com/coronasafe/care/pull/1748#issuecomment-1842536902
Shared a comment on a pull request in coronasafe/care on November 3, 2023 at 3:54:13 PM
@Pranshu1902 the test are not catching the changes, try to assert the error message response
https://github.com/coronasafe/care/pull/1157#issuecomment-1792693637
Shared a comment on a pull request in coronasafe/care on November 3, 2023 at 7:13:31 AM
> When removing a duty staff user from asset location, those records are not set to `deleted=True` instead it's permanently deleted right? > > cc: @sainak yeah we can use a custom through model cc: @vigneshhari
https://github.com/coronasafe/care/pull/1689#issuecomment-1791973391
Shared a comment on an issue in coronasafe/care_fe on October 26, 2023 at 10:06:55 AM
https://care-staging.ohc.network/assets/
https://github.com/coronasafe/care_fe/issues/6500#issuecomment-1780810603
Shared a comment on a pull request in coronasafe/care on October 20, 2023 at 9:18:47 AM
The solution would be to index the bed names and before creating beds do a lookup of any existing beds with names but creating unique index can cause conflict, so this is a blocker @vigneshhari what are your thoughts?
https://github.com/coronasafe/care/pull/1530#issuecomment-1772378468
Shared a comment on a pull request in coronasafe/care on October 20, 2023 at 9:11:44 AM
@Pranshu1902 delete and recreate the migrations
https://github.com/coronasafe/care/pull/1505#issuecomment-1772367249
Shared a comment on a pull request in coronasafe/care on October 20, 2023 at 6:49:47 AM
> @sainak should we move the loading of icd11 to in-memory for tests to somewhere else? > > or is it fine for now since we'll soon rewrite how we populate icd11/medibase to in memory in the first place? it's fine for now
https://github.com/coronasafe/care/pull/1673#issuecomment-1772180854
Shared a comment on a pull request in coronasafe/care_fe on October 16, 2023 at 12:49:06 PM
https://deploy-preview-6457--care-egov-staging.netlify.app/facility/9bc6e011-3ab6-4aec-ba89-f4cda407fa13/assets/7fd1ec81-f537-421f-8ec0-1bee1c675748/configure accessible on Palakkad
https://github.com/coronasafe/care_fe/pull/6457#issuecomment-1764416574
Shared a comment on a pull request in coronasafe/care on October 3, 2023 at 8:24:31 AM
> @sainak is this ready to review or still pending changes? This was a kind of poc but, you can review and give feedbacks or suggest better ways cc: @vigneshhari
https://github.com/coronasafe/care/pull/1349#issuecomment-1744454729
Shared a comment on an issue in coronasafe/care_fe on October 3, 2023 at 6:35:07 AM
This will be handled by https://github.com/coronasafe/care/issues/895
https://github.com/coronasafe/care_fe/issues/1091#issuecomment-1744296431
Shared a comment on a pull request in coronasafe/care_fe on September 20, 2023 at 4:08:43 PM
@Bhavik-ag instead of refreshing the whole page, use the notes object from webpush https://github.com/coronasafe/care/pull/1553/files#diff-9f3ecbbd375e572272af9a71bc539ebf66215f3c81e3f8ed8ee8ced1286f77daR626
https://github.com/coronasafe/care_fe/pull/6149#issuecomment-1728041271
Shared a comment on a pull request in coronasafe/care on September 19, 2023 at 4:07:48 PM
@Ashesh3 add tests for the retrieve method
https://github.com/coronasafe/care/pull/1625#issuecomment-1725981171
Shared a comment on an issue in coronasafe/care on September 19, 2023 at 3:32:35 PM
closed in https://github.com/coronasafe/care/pull/1592
https://github.com/coronasafe/care/issues/1327#issuecomment-1725900216
Shared a comment on a pull request in coronasafe/care on September 12, 2023 at 7:25:27 AM
@dependabot ignore this major version
https://github.com/coronasafe/care/pull/1604#issuecomment-1715148460
Shared a comment on a pull request in coronasafe/care on September 11, 2023 at 4:26:01 PM
> @sainak just a thought, since we are repeating the sentry step multiple times, we could consider using YAML reusable blocks with overrides right (for `environment`)? > > eg: [linode.com/docs/guides/yaml-anchors-aliases-overrides-extensions/#yaml-overrides](https://www.linode.com/docs/guides/yaml-anchors-aliases-overrides-extensions/#yaml-overrides) I'm planning to switch the deployment definitions with reusable workflow calls
https://github.com/coronasafe/care/pull/1590#issuecomment-1714212584
Shared a comment on a pull request in coronasafe/care on September 8, 2023 at 3:22:15 PM
@rithviknishad set blank=True here https://github.com/coronasafe/care/pull/1567/files#diff-5f86d36c9415c5c40e8820c1e4f86aa28dd713d6a74a1adc1f387598b8b542b9R147
https://github.com/coronasafe/care/pull/1586#issuecomment-1711841357
Shared a comment on an issue in coronasafe/leaderboard on September 7, 2023 at 9:53:50 AM
https://vercel.com/docs/functions/edge-functions/og-image-generation
https://github.com/coronasafe/leaderboard/issues/127#issuecomment-1709858928
Shared a comment on a pull request in coronasafe/teleicu_middleware on September 4, 2023 at 7:32:25 AM
@Amanshenoy008 can you resolve the conflicts
https://github.com/coronasafe/teleicu_middleware/pull/74#issuecomment-1704757959
Shared a comment on a pull request in coronasafe/care_fe on September 1, 2023 at 7:06:15 AM
`test -f .env.local && dotenv .env.local`
https://github.com/coronasafe/care_fe/pull/6201#issuecomment-1702273294
Shared a comment on a pull request in coronasafe/care_fe on September 1, 2023 at 6:58:59 AM
We should add direnv as well
https://github.com/coronasafe/care_fe/pull/6201#issuecomment-1702265405
Shared a comment on a pull request in coronasafe/care_fe on September 1, 2023 at 6:33:45 AM
@rithviknishad https://github.com/coronasafe/care_fe/blob/c3f3ce52293addc028246c3ae4831b3c841354b1/.github/workflows/cypress.yaml#L39
https://github.com/coronasafe/care_fe/pull/6201#issuecomment-1702240109
Shared a comment on an issue in coronasafe/care_fe on August 30, 2023 at 7:10:38 AM
@nihal467 we can close this as we have a detailed issue filed for this: #1091
https://github.com/coronasafe/care_fe/issues/928#issuecomment-1698618273
Shared a comment on a pull request in coronasafe/care on August 29, 2023 at 11:40:33 AM
needs test
https://github.com/coronasafe/care/pull/1565#issuecomment-1697278876
Shared a comment on a pull request in coronasafe/care on August 25, 2023 at 9:30:06 AM
@rithviknishad can we switch to https://django-filter.readthedocs.io/en/stable/ref/filters.html#numberfilter
https://github.com/coronasafe/care/pull/1560#issuecomment-1693065981
Shared a comment on a pull request in coronasafe/care on August 24, 2023 at 2:26:48 PM
@vigneshhari this pr is good to merge, we got ops approval
https://github.com/coronasafe/care/pull/1507#issuecomment-1691789883
Shared a comment on a pull request in coronasafe/care on August 24, 2023 at 6:22:54 AM
@Ashesh3 migrations needs to updated
https://github.com/coronasafe/care/pull/1277#issuecomment-1691072995
Shared a comment on an issue in coronasafe/care on August 22, 2023 at 11:42:26 AM
fixed in #1524
https://github.com/coronasafe/care/issues/934#issuecomment-1688024542
Shared a comment on a pull request in coronasafe/care on August 8, 2023 at 6:54:35 PM
> Switch to [docs.djangoproject.com/en/4.2/ref/models/fields/#positiveintegerfield](https://docs.djangoproject.com/en/4.2/ref/models/fields/#positiveintegerfield) @Pranshu1902 make sure to handle negative values while migrating
https://github.com/coronasafe/care/pull/1505#issuecomment-1670131330
Shared a comment on an issue in coronasafe/care on August 8, 2023 at 11:59:15 AM
@Jrl0, @aparnacoronasafe please check the state of this issue once more and reopen if it still exists
https://github.com/coronasafe/care/issues/985#issuecomment-1669477414
Shared a comment on a pull request in coronasafe/care on August 4, 2023 at 5:35:23 AM
Tests failing as expected
https://github.com/coronasafe/care/pull/1509#issuecomment-1665038008
Shared a comment on a pull request in coronasafe/care on August 4, 2023 at 5:28:17 AM
> No migrations???? yes migrations will be required
https://github.com/coronasafe/care/pull/1505#issuecomment-1665033147
Shared a comment on an issue in coronasafe/care on August 3, 2023 at 1:36:05 PM
#1475
https://github.com/coronasafe/care/issues/1483#issuecomment-1663998552
Shared a comment on an issue in coronasafe/care on August 3, 2023 at 7:18:11 AM
> > @sainak I tried running a local pre-commit to run `./manage.py makemigrations --check --dry-run` > > > > This works fine when running via CLI (`git commit -m ...`), but fails to run when commiting from vscode git extension. Failed to import Django. > > Does the vscode git command run outside the context of the current virtual environment, hence the failing import? Yes precommit creates virtual env for each hook and uses that by default If you want to run checks with local packages use the system type for hooks
https://github.com/coronasafe/care/issues/1499#issuecomment-1663422448
Shared a comment on an issue in coronasafe/care on August 2, 2023 at 5:43:52 PM
There is a precommit hook for it We can add that
https://github.com/coronasafe/care/issues/1499#issuecomment-1662683885
Shared a comment on an issue in coronasafe/care on July 26, 2023 at 8:13:52 AM
Sure will do it
https://github.com/coronasafe/care/issues/812#issuecomment-1651198884
Shared a comment on a pull request in coronasafe/care on July 24, 2023 at 7:54:10 AM
Closing in favor of #1469
https://github.com/coronasafe/care/pull/1285#issuecomment-1647399453
Shared a comment on an issue in coronasafe/care on July 21, 2023 at 4:48:06 PM
fixed with django 4.2
https://github.com/coronasafe/care/issues/1015#issuecomment-1645981085
Shared a comment on an issue in coronasafe/care on July 21, 2023 at 4:47:24 PM
completed in #1454
https://github.com/coronasafe/care/issues/1108#issuecomment-1645980241
Shared a comment on an issue in coronasafe/care on July 21, 2023 at 4:46:12 PM
We can use `runserver_plus --print-sql` or `ddt` or `silk_profiler` locally and for deployments we can check sentry
https://github.com/coronasafe/care/issues/1321#issuecomment-1645978677
Shared a comment on an issue in coronasafe/care on July 21, 2023 at 4:42:34 PM
we have load_dummy_data command to bootstrap database for testing
https://github.com/coronasafe/care/issues/920#issuecomment-1645973538
Shared a comment on an issue in coronasafe/care on July 21, 2023 at 4:37:53 PM
to be implemented by #968
https://github.com/coronasafe/care/issues/486#issuecomment-1645966331
Shared a comment on a pull request in coronasafe/care on July 20, 2023 at 6:10:20 PM
@GeekGawd resolve confilicts
https://github.com/coronasafe/care/pull/1427#issuecomment-1644373480
Shared a comment on a pull request in coronasafe/care on July 20, 2023 at 5:36:21 PM
> Migrations could be squashed on it
https://github.com/coronasafe/care/pull/1457#issuecomment-1644325764
Shared a comment on a pull request in coronasafe/care on July 20, 2023 at 4:27:40 PM
@cp-Coder add test cases
https://github.com/coronasafe/care/pull/1058#issuecomment-1644230172
Shared a comment on an issue in coronasafe/care on July 19, 2023 at 1:35:26 PM
@vigneshhari the current default soft limit is 30 minutes this is because a there are a lot of records in staging
https://github.com/coronasafe/care/issues/1462#issuecomment-1642097769
Shared a comment on a pull request in coronasafe/care on July 19, 2023 at 7:02:57 AM
Closing in favor of #1447
https://github.com/coronasafe/care/pull/1399#issuecomment-1641529098
Shared a comment on a pull request in coronasafe/care on July 19, 2023 at 7:00:56 AM
@Ashesh3 resolve the conflicts
https://github.com/coronasafe/care/pull/1221#issuecomment-1641526697
Shared a comment on a pull request in coronasafe/care on July 17, 2023 at 4:51:14 PM
@Ashesh3 add tests for api endpoints
https://github.com/coronasafe/care/pull/1454#issuecomment-1638513504
Shared a comment on a pull request in coronasafe/care on July 17, 2023 at 6:37:12 AM
@Bhavik-ag this serializer is being used at many places and adding the home_facility field will cause nested query calls everywhere, so it would be better to add a new field in `PatientNotesSerializer` called `created_by_local_user` which could be annotated on the queryset.
https://github.com/coronasafe/care/pull/1445#issuecomment-1637459842
Shared a comment on a pull request in coronasafe/care on July 7, 2023 at 9:24:06 AM
Yeah we are using the id field for it right
https://github.com/coronasafe/care/pull/1439#issuecomment-1625122404
Shared a comment on an issue in coronasafe/care on July 6, 2023 at 8:26:50 PM
You can take a look at the API requests in the test cases for password reset and update the endpoints accordingly
https://github.com/coronasafe/care/issues/1434#issuecomment-1624268357
Shared a comment on a pull request in coronasafe/care on July 6, 2023 at 8:22:50 PM
> @sainak I have a couple of questions: > - referred_to_external should accept only strings/null values right? And not integer etc. Yes > - Should I create separate branch for tests or push them here only? You can include them in this pr
https://github.com/coronasafe/care/pull/1430#issuecomment-1624264311
Shared a comment on a pull request in coronasafe/care on July 6, 2023 at 8:08:56 PM
Migrations tested on latest staging snapshot
https://github.com/coronasafe/care/pull/1439#issuecomment-1624249076
Shared a comment on a pull request in coronasafe/teleicu_middleware on July 6, 2023 at 1:00:47 PM
Good work @Amanshenoy008, there is one change needed, we prefer npm for package management, the yarn.lock was accidentally added to the repo which will be removed, can you also revert the changes done to yarn.lock and update package-lock.json instead
https://github.com/coronasafe/teleicu_middleware/pull/74#issuecomment-1623640339
Shared a comment on a pull request in coronasafe/care on July 6, 2023 at 12:35:20 PM
> This filter would work only when `is_active=False` right? we have a `is_active` filter for it
https://github.com/coronasafe/care/pull/1440#issuecomment-1623604678
Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 2:30:46 PM
@cp-Coder can you rebase this on the latest master branch
https://github.com/coronasafe/care/pull/1073#issuecomment-1621882838
Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 1:50:51 PM
can we add a basic image tool to crop and resize the image, and then just check the size at the backend @vigneshhari
https://github.com/coronasafe/care/pull/1106#issuecomment-1621804678
Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 1:46:03 PM
@cp-Coder we already have a field `taken_at` in the Daily round model, which should be enough, we can show the taken_at time if present on the frontend cc: @gigincg
https://github.com/coronasafe/care/pull/1167#issuecomment-1621794274
Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 1:13:49 PM
> @sainak @vigneshhari And why is reffered_to deprecated? it was changed in #715, but I couldn't find the reference of the issue that requested the change
https://github.com/coronasafe/care/pull/1430#issuecomment-1621733943
Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 11:42:47 AM
Looks good, just add a todo [here](https://github.com/coronasafe/care/pull/1427/files#diff-e81f72006459ce5c1245abbb101a1345e909b90ec7211585c02a44a232e0a0cfR63) to remove when remove serializers are updated
https://github.com/coronasafe/care/pull/1427#issuecomment-1621591521
Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 9:56:11 AM
> @sainak can you separate it out, we can merge it right away. on it
https://github.com/coronasafe/care/pull/1431#issuecomment-1621430744
Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 9:54:17 AM
@GeekGawd tests failing
https://github.com/coronasafe/care/pull/1427#issuecomment-1621427199
Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 8:22:14 AM
@sachdevavaibhav can you add some tests to validate whether this field is accepting correct inputs
https://github.com/coronasafe/care/pull/1430#issuecomment-1621273132
Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 8:02:56 AM
@cp-Coder Split up the test cases, this will help better in tracking the issue and speed up tests
https://github.com/coronasafe/care/pull/1431#issuecomment-1621242341
Shared a comment on a pull request in coronasafe/care on July 4, 2023 at 5:56:46 PM
@vigneshhari this is a good time to migrate and remove `referred_to` foreign key https://github.com/coronasafe/care/blob/b0eefd152511a390f759b41dac0cb7e27c9ee117/care/facility/models/patient_consultation.py#L92-L98
https://github.com/coronasafe/care/pull/1430#issuecomment-1620585772
Shared a comment on a pull request in coronasafe/care on July 4, 2023 at 4:45:37 PM
@GeekGawd the changes look good, can you verify and remove the AssetSerializer from other places as well Note: the AssetSerializer is also used to fetch data [here](https://github.com/coronasafe/care_dashboard/blob/master/src/pages/asset/AssetDetails.tsx#L31) using the `public/asset/` endpoint, this looks like it could be handled by the AssetDetailSerializer and for the tests you can add and validate more data.
https://github.com/coronasafe/care/pull/1427#issuecomment-1620528147
Shared a comment on an issue in coronasafe/care on July 1, 2023 at 2:53:32 PM
@mathew-alex has increased the number of scaling replicas to 4
https://github.com/coronasafe/care/issues/1421#issuecomment-1615949074
Shared a comment on a pull request in coronasafe/care on July 1, 2023 at 12:27:36 PM
@khavinshankar this issue is present in the latest version of fhir.resources, this compatibility check needs to be done on the upstream itself, plese verify and file an issue on fhir.resources repo if necessary
https://github.com/coronasafe/care/pull/1418#issuecomment-1615885360
Shared a comment on a pull request in coronasafe/care on June 24, 2023 at 1:15:44 PM
@vigneshhari i've updated the migrations
https://github.com/coronasafe/care/pull/1303#issuecomment-1605488339
Shared a comment on a pull request in coronasafe/care on June 14, 2023 at 12:55:59 PM
are these test cases?
https://github.com/coronasafe/care/pull/1368#issuecomment-1591153134
Shared a comment on an issue in coronasafe/care on June 14, 2023 at 5:40:31 AM
> > @Yash12Bansal Please link the issue to the PR you are referring to > > also please tell me where and what changes were made for Chromium support. I cannot find them in new commits @gigincg https://github.com/coronasafe/care/compare/912bd36511c95a640101363d45f5c009016020f4...a50188884326ed31e891bf03ecdaf41a8311f36b
https://github.com/coronasafe/care/issues/812#issuecomment-1590504429
Shared a comment on a pull request in coronasafe/care_fe on June 5, 2023 at 4:21:36 AM
> @sainak it's already in the OHC domain right? I do see that I've changed it from the diff. Sorry, I mixed up the branches
https://github.com/coronasafe/care_fe/pull/5587#issuecomment-1576021298
Shared a comment on a pull request in coronasafe/care_fe on June 4, 2023 at 6:07:00 PM
@rithviknishad update the vite config as well https://github.com/coronasafe/care_fe/blob/5982c94c8e210f9b3e6e0168989b5db2d89e39c8/vite.config.ts#L77
https://github.com/coronasafe/care_fe/pull/5587#issuecomment-1575660011
Shared a comment on a pull request in coronasafe/care on June 1, 2023 at 3:43:20 PM
In a through model the `on_delete` should be set to `models.CASCADE`, was there any specific reason to set it to `models.PROTECT` ? @vigneshhari
https://github.com/coronasafe/care/pull/1325#issuecomment-1572297032
Shared a comment on a pull request in coronasafe/care on June 1, 2023 at 11:56:59 AM
@yaswanthsaivendra there are no changes in this pr you should create a new branch for every pull request
https://github.com/coronasafe/care/pull/1310#issuecomment-1571914438
Shared a comment on a pull request in coronasafe/care on May 27, 2023 at 4:28:26 AM
@yaswanthsaivendra in patient consultation view you can prefetch `assigned_to__skills`
https://github.com/coronasafe/care/pull/1310#issuecomment-1565202103
Shared a comment on an issue in coronasafe/care_fe on May 25, 2023 at 11:59:35 AM
@SaiRev0 @MayaSatishRao it's recommended to use a unix environment for developing care. Can you try running the project in wsl.
https://github.com/coronasafe/care_fe/issues/5515#issuecomment-1562779339
Shared a comment on an issue in coronasafe/care on May 24, 2023 at 5:24:20 AM
closed in #924
https://github.com/coronasafe/care/issues/1086#issuecomment-1560475377
Shared a comment on an issue in coronasafe/care on May 24, 2023 at 4:18:37 AM
resolved in https://github.com/coronasafe/care/pull/924
https://github.com/coronasafe/care/issues/911#issuecomment-1560436674
Shared a comment on an issue in coronasafe/care on May 20, 2023 at 10:23:25 AM
closed by #1252
https://github.com/coronasafe/care/issues/1205#issuecomment-1555881562
Shared a comment on an issue in coronasafe/care_fe on April 24, 2023 at 11:15:50 AM
> > @Ashesh3 @nihal467 Please expedite it > > @sainak Are the required changes done on the backend? Is this issue ready to be worked on at the front end? Both the backend and frontend changes are done and they are under review
https://github.com/coronasafe/care_fe/issues/5237#issuecomment-1519942743
Shared a comment on an issue in coronasafe/care_fe on April 20, 2023 at 12:30:24 PM
@aparnacoronasafe @Ashesh3 I moved the last point from #5302 to this issue as it requires frontend changes as well
https://github.com/coronasafe/care_fe/issues/5300#issuecomment-1516244689
Shared a comment on a pull request in coronasafe/care on April 14, 2023 at 12:35:37 PM
required for coronasafe/care_fe#5246
https://github.com/coronasafe/care/pull/1252#issuecomment-1508441173
Shared a comment on an issue in coronasafe/care_fe on April 12, 2023 at 8:29:45 AM
> @sainak can you provide an ETA Will be done by thursday eod
https://github.com/coronasafe/care_fe/issues/5246#issuecomment-1504874429
Shared a comment on a pull request in coronasafe/care_fe on March 28, 2023 at 7:01:44 AM
@gigincg @mathew-alex the PR is ready to be merged in develop with one breaking change that is the env variable `PUBLIC_URL` needs to be renamed to `REACT_PUBLIC_URL`
https://github.com/coronasafe/care_fe/pull/5140#issuecomment-1486321207
Shared a comment on a pull request in coronasafe/care_fe on March 24, 2023 at 9:56:28 AM
@rithviknishad please update the ci for storybook the build commnad is `build:storybook` and the dist folder is `build-storybook` also can you verify if the service worker is working correctly?
https://github.com/coronasafe/care_fe/pull/5140#issuecomment-1482534433
Shared a comment on a pull request in coronasafe/care_fe on March 23, 2023 at 10:16:49 AM
@gigincg changes to deployment config won't be needed as internally npm run build is calling vite You can see the latest deploy preview, the components are working fine now but the css file is quite large as compared to cra builds, service worker might be broken, and I'm working on fixing the storybook
https://github.com/coronasafe/care_fe/pull/5140#issuecomment-1480928059
Shared a comment on a pull request in coronasafe/care_fe on March 22, 2023 at 6:40:45 AM
@rithviknishad the issue is fixed now. We will have to be careful with what external libraries we use.
https://github.com/coronasafe/care_fe/pull/5140#issuecomment-1478991971
Shared a comment on a pull request in coronasafe/care_fe on March 22, 2023 at 3:53:30 AM
@rithviknishad the components contaning the phone number field break while previewing a production build, I'm not able to pin point the actual issue but here's the error that I got ``` Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: [missing argument].[missing argument] ``` ![image](https://user-images.githubusercontent.com/46787056/226797774-98d91a15-d127-4ebb-b63c-3021f1c7c23e.png) please check this issue
https://github.com/coronasafe/care_fe/pull/5140#issuecomment-1478883709
Shared a comment on a pull request in coronasafe/care on February 23, 2023 at 6:27:23 PM
> @sainak can you recheck the regex, even the valid URLs are failing > > ![image](https://user-images.githubusercontent.com/29787772/220268219-9815dfc2-ed52-4171-b8ba-af33f372ee6f.png) That is the intended behavior because we use the same value for websockets in frontend as well, so saving it with scheme breaks the frontend requirement
https://github.com/coronasafe/care/pull/1199#issuecomment-1442238242
Shared a comment on an issue in coronasafe/care_fe on February 21, 2023 at 6:33:07 AM
This can be validated on the frontend itself the regex is `^(?!https?:\/\/)[a-zA-Z0-9_-]+(\.[a-zA-Z0-9_-]+)*\.[a-zA-Z]{2,}$`
https://github.com/coronasafe/care_fe/issues/4857#issuecomment-1437926851
Shared a comment on a pull request in coronasafe/care on January 12, 2023 at 10:40:59 AM
> Are we no longer pointing to local if a middleware address is not present? This change would give more info to the admin about whats going wrong. we were pointing it to local for testing, we can still set the value to localhost in the configurations
https://github.com/coronasafe/care/pull/1144#issuecomment-1380134129
Shared a comment on an issue in coronasafe/care_fe on December 13, 2022 at 1:58:15 PM
@aparnacoronasafe @gigincg is this still needed?
https://github.com/coronasafe/care_fe/issues/2759#issuecomment-1348622293
Shared a comment on an issue in coronasafe/care on December 13, 2022 at 1:15:29 PM
I think this is what you are looking for: ```shell curl -X 'GET' \ 'https://api.data.gov.in/resource/5c2f62fe-5afa-4119-a499-fec9d604d5bd?api-key=579b464db66ec23bdd00000115f15ab447b749065c58768a487d76bb&format=json&filters%5Bpincode%5D=400001' \ -H 'accept: application/json' ``` ref: https://data.gov.in/apis/5c2f62fe-5afa-4119-a499-fec9d604d5bd the public api key is good for upto 10 records per request so we can directly use this in the frontend
https://github.com/coronasafe/care/issues/1107#issuecomment-1348519980
Shared a comment on an issue in coronasafe/care on December 4, 2022 at 1:17:18 PM
done https://github.com/coronasafe/care/labels/stale
https://github.com/coronasafe/care/issues/1113#issuecomment-1336409677
Shared a comment on a pull request in coronasafe/qr-generator on November 15, 2022 at 6:16:27 PM
It became so fast that we are skipping some images
https://github.com/coronasafe/qr-generator/pull/7#issuecomment-1315696347
Shared a comment on a pull request in coronasafe/care on November 13, 2022 at 12:46:40 PM
@khavinshankar please make sure the lint checks are passing
https://github.com/coronasafe/care/pull/1091#issuecomment-1312722833
Shared a comment on a pull request in coronasafe/care on September 10, 2022 at 6:28:40 PM
> Let's not give the internal name out, our goal here was to break the relationship between the file name and the patient in the unlikely case our buckets are attacked. Let's serialize only the extension. yeah I thought returning the extension would be a better solution do you have any suggestions for implementing that? I have 2 implementations in mind - making the query return the extension - make a model method to return extension or file type
https://github.com/coronasafe/care/pull/1019#issuecomment-1242782897
Shared a comment on a pull request in coronasafe/care on September 9, 2022 at 10:11:37 AM
@Pragati1610 rebase your changes on care/master branch
https://github.com/coronasafe/care/pull/1016#issuecomment-1241778703
Shared a comment on a pull request in coronasafe/care on September 9, 2022 at 9:07:52 AM
duplicate of #1013
https://github.com/coronasafe/care/pull/1017#issuecomment-1241709996
Shared a comment on a pull request in coronasafe/care on September 5, 2022 at 8:02:45 PM
https://github.com/coronasafe/care_fe/issues/2171
https://github.com/coronasafe/care/pull/1003#issuecomment-1237426554
Shared a comment on an issue in coronasafe/leaderboard on September 3, 2022 at 8:45:14 AM
@Ritesh-Aggarwal yes thats the current behaviour.
https://github.com/coronasafe/leaderboard/issues/79#issuecomment-1236076980
Shared a comment on a pull request in coronasafe/care on July 27, 2022 at 6:10:05 AM
@Marmik2003 can you check this pr again, it's breaking https://sentry.io/organizations/coronasafe-network/issues/3453690266/?project=5182842&query=is%3Aunresolved
https://github.com/coronasafe/care/pull/791#issuecomment-1196304593
Shared a comment on a pull request in coronasafe/care on July 12, 2022 at 7:05:09 PM
merging with #924
https://github.com/coronasafe/care/pull/841#issuecomment-1182298909
Shared a comment on a pull request in coronasafe/care on July 12, 2022 at 7:03:39 PM
superseded by #924
https://github.com/coronasafe/care/pull/807#issuecomment-1182295414
Shared a comment on an issue in coronasafe/care on June 25, 2022 at 1:08:38 PM
> Is facility icon the same as facility cover image or different: #791 ? @Marmik2003 this api is needed to give out the summary for indicators mentioned in care_fe#2759
https://github.com/coronasafe/care/issues/881#issuecomment-1166281364
Shared a comment on an issue in coronasafe/care on June 25, 2022 at 12:52:11 PM
@gigincg it's a nice to have, chrome is closed sourced, vignesh tried using chromium but that didnt work either.
https://github.com/coronasafe/care/issues/812#issuecomment-1166277839
Shared a comment on an issue in coronasafe/care on June 25, 2022 at 1:08:38 PM
> Is facility icon the same as facility cover image or different: #791 ? @Marmik2003 this api is needed to give out the summary for indicators mentioned in care_fe#2759
https://github.com/coronasafe/care/issues/881#issuecomment-1166281364
Shared a comment on an issue in coronasafe/care on June 25, 2022 at 12:52:11 PM
@gigincg it's a nice to have, chrome is closed sourced, vignesh tried using chromium but that didnt work either.
https://github.com/coronasafe/care/issues/812#issuecomment-1166277839
Shared a comment on a pull request in coronasafe/care on June 24, 2022 at 5:00:11 AM
> Also this field is also present in the template. So do I have to delete from template too? we are not using template views anymore and it is planned to be removed so its fine to leave it as it is.
https://github.com/coronasafe/care/pull/882#issuecomment-1165197102
Shared a comment on a pull request in coronasafe/care on June 24, 2022 at 5:00:11 AM
> Also this field is also present in the template. So do I have to delete from template too? we are not using template views anymore and it is planned to be removed so its fine to leave it as it is.
https://github.com/coronasafe/care/pull/882#issuecomment-1165197102
Shared a comment on a pull request in coronasafe/care on June 23, 2022 at 6:22:12 PM
@skks1212 whats [this](https://github.com/coronasafe/care/pull/874/files#diff-af4f6a3a22f741955002ce0a55764940a9082a84d779e3db3c80b0f406ce4755) file for?
https://github.com/coronasafe/care/pull/874#issuecomment-1164731144
Shared a comment on a pull request in coronasafe/care on June 23, 2022 at 6:22:12 PM
@skks1212 whats [this](https://github.com/coronasafe/care/pull/874/files#diff-af4f6a3a22f741955002ce0a55764940a9082a84d779e3db3c80b0f406ce4755) file for?
https://github.com/coronasafe/care/pull/874#issuecomment-1164731144
Shared a comment on an issue in coronasafe/care on June 9, 2022 at 7:22:41 PM
> Another thing worth mentioning is that we are being spammed by requests with unknown hosts which triggers `django.security.DisallowedHost` events on sentry these events are not worth tracking and they only consume events bandwidth Its possible to ignore these events from the deployment itself. @vigneshhari whats your take on this?
https://github.com/coronasafe/care/issues/844#issuecomment-1151527610
Shared a comment on an issue in coronasafe/care on June 9, 2022 at 7:07:48 PM
@JAIMIN-CHOKHAWALA https://github.com/coronasafe/care_fe/pull/2665 will update the status message
https://github.com/coronasafe/care/issues/842#issuecomment-1151501089
Shared a comment on an issue in coronasafe/care on June 9, 2022 at 6:33:31 PM
closing this issue as it's being tracked in https://github.com/coronasafe/care_fe/issues/2664
https://github.com/coronasafe/care/issues/842#issuecomment-1151468957
Shared a comment on a pull request in coronasafe/care on June 8, 2022 at 1:07:59 PM
@skks1212 @Marmik2003 we can integrate the test step in the deployment workflow https://github.com/coronasafe/care/blob/master/.github/workflows/deployment.yaml#L54-L56
https://github.com/coronasafe/care/pull/835#issuecomment-1149893118
Shared a comment on a pull request in coronasafe/care on June 7, 2022 at 2:38:11 PM
> Looks good, i think it is ok if we don't remove sphinx from local requirements as some developers might also want to change the docs after few major changes! I've refrenced the docs.txt requirements in local.txt so sphinx will also get installed in local environments
https://github.com/coronasafe/care/pull/826#issuecomment-1148765297
Shared a comment on a pull request in coronasafe/care on June 7, 2022 at 1:45:28 PM
@vigneshhari add https://pre-commit.ci/ to the repo
https://github.com/coronasafe/care/pull/836#issuecomment-1148698681
Shared a comment on an issue in coronasafe/care on June 7, 2022 at 11:20:15 AM
closed by 9e03d628f6bc657e02b2d266fea8f5dbf215a3a8
https://github.com/coronasafe/care/issues/667#issuecomment-1148532949
Shared a comment on an issue in coronasafe/care on June 7, 2022 at 11:11:44 AM
closed by #715
https://github.com/coronasafe/care/issues/674#issuecomment-1148524808
Shared a comment on an issue in coronasafe/care on June 7, 2022 at 11:00:23 AM
Closing in favour of #794
https://github.com/coronasafe/care/issues/488#issuecomment-1148514653
Shared a comment on a pull request in coronasafe/care on June 6, 2022 at 9:42:22 PM
@Marmik2003 there are a lot of changes that I want to make, I would suggest pushing the changes to a new branch in care itself `marmik2003/feat/uset-tests` and create a draft PR from that branch
https://github.com/coronasafe/care/pull/824#issuecomment-1147961631
Shared a comment on a pull request in coronasafe/care on June 6, 2022 at 4:03:01 PM
This change would be a 2 step process 1. Migrating the location field to normal fields 2. removing references of the location filed from older migrations (0015, 0017)
https://github.com/coronasafe/care/pull/822#issuecomment-1147615420
Shared a comment on an issue in coronasafe/care on June 4, 2022 at 1:54:57 PM
> > > We can but let it be there. Since we have multiple instances being deployed to > > > > > > @tomahawk-pilot I'm asking auto deployments only for Staging-egov, If the review requirement could be removed for `Staging-egov` environment the FE development and testing won't be blocked by pending deployments. > > Feel free to remove the review requirements for staging egov environment I don't have the privileges for it.
https://github.com/coronasafe/care/issues/792#issuecomment-1146617067
Shared a comment on an issue in coronasafe/care on June 4, 2022 at 10:09:40 AM
> We can but let it be there. Since we have multiple instances being deployed to @tomahawk-pilot I'm asking auto deployments only for Staging-egov, If the review requirement could be removed for `Staging-egov` environment the FE development and testing won't be blocked by pending deployments.
https://github.com/coronasafe/care/issues/792#issuecomment-1146582385
Shared a comment on an issue in coronasafe/care on June 3, 2022 at 9:04:02 PM
It's waiting for deployment https://github.com/coronasafe/care/deployments/activity_log?environment=Staging-egov @mathew-alex @tomahawk-pilot can you update the staging to deploy automatically?
https://github.com/coronasafe/care/issues/792#issuecomment-1146360771
Shared a comment on an issue in coronasafe/care on June 3, 2022 at 8:46:44 PM
The asset_class expects an integer Exception: https://sentry.io/organizations/coronasafe-network/issues/3306110898/
https://github.com/coronasafe/care/issues/792#issuecomment-1146349157
Shared a comment on an issue in coronasafe/care on June 3, 2022 at 8:39:53 PM
> Which site Not sure but it was cause by this worker `care-staging-celery-worker-7dfcc76ffd-gc8f8`
https://github.com/coronasafe/care/issues/816#issuecomment-1146344671
Shared a comment on an issue in coronasafe/care_fe on June 3, 2022 at 8:35:39 PM
tracked in https://github.com/coronasafe/care/issues/816
https://github.com/coronasafe/care_fe/issues/2298#issuecomment-1146341427
Shared a comment on an issue in coronasafe/care on June 3, 2022 at 8:34:01 PM
Cc @tomahawk-pilot @mathew-alex
https://github.com/coronasafe/care/issues/816#issuecomment-1146340300
Shared a comment on a pull request in coronasafe/care_fe on May 30, 2022 at 4:43:27 PM
@Pragati1610 delete the package-lock from your branch as we have started using yarn.
https://github.com/coronasafe/care_fe/pull/2590#issuecomment-1141340249
Shared a comment on a pull request in coronasafe/care on May 27, 2022 at 5:20:47 PM
@rabilrbl recreate the migrations
https://github.com/coronasafe/care/pull/769#issuecomment-1139827654
Shared a comment on a pull request in coronasafe/care on May 27, 2022 at 11:22:54 AM
There are relevent changes only the force push was caused by rebasing the branch to update the migrations
https://github.com/coronasafe/care/pull/746#issuecomment-1139527875
Shared a comment on an issue in coronasafe/care on May 25, 2022 at 8:21:37 PM
@siddnikh we can just fix the `UserSerializer` update method to handle password field changes, also make the password a writeonly field, by doing this we can just PATCH the password, I don't think there is a requirement for a separate endpoint.
https://github.com/coronasafe/care/issues/779#issuecomment-1137811990
Shared a comment on a pull request in coronasafe/care on May 25, 2022 at 7:55:10 PM
This should be good to merge as now we are using a single docker image.
https://github.com/coronasafe/care/pull/724#issuecomment-1137789177
Shared a comment on a pull request in coronasafe/care_fe on May 25, 2022 at 8:27:57 AM
@cp-Coder update https://www.npmjs.com/package/rescript-material-ui as well
https://github.com/coronasafe/care_fe/pull/2427#issuecomment-1136948602
Shared a comment on a pull request in coronasafe/care on May 22, 2022 at 4:47:09 PM
Ignore the review it was for a test
https://github.com/coronasafe/care/pull/740#issuecomment-1133933110
Shared a comment on a pull request in coronasafe/care on May 22, 2022 at 9:36:29 AM
what we can do is rename the min_quantity field to default_min_quantity in the response and decide the is_low based on the `FacilityInventoryMinQuantity`
https://github.com/coronasafe/care/pull/772#issuecomment-1133856645
Shared a comment on a pull request in coronasafe/care on May 22, 2022 at 9:34:12 AM
Its possible to do it on the BE but the min_quantity will still be redundant atlest in `FacilityInventoryLogSerializer`
https://github.com/coronasafe/care/pull/772#issuecomment-1133856253
Shared a comment on a pull request in coronasafe/care on May 22, 2022 at 9:31:10 AM
> @sainAk the min_quantity can stay and maybe FE can ignore the flag if the min_quantity value is not present, or we can add that logic into our flagging system. Sure I'll apply the changes
https://github.com/coronasafe/care/pull/772#issuecomment-1133855741
Shared a comment on a pull request in coronasafe/care on May 22, 2022 at 9:28:28 AM
For now I'm only building the docker image, after we add localstack we can switch over to compose for testing.
https://github.com/coronasafe/care/pull/768#issuecomment-1133855283
Shared a comment on an issue in coronasafe/care_fe on May 21, 2022 at 8:35:12 PM
This issue is caused because `Asset QR Code ID` is sending blank string which causes integrity error @rabilrbl do you want to take up this issue to handle it on the FE?
https://github.com/coronasafe/care_fe/issues/2422#issuecomment-1133764807
Shared a comment on an issue in coronasafe/care_fe on May 20, 2022 at 7:29:48 PM
@gigincg this is not reproduceable locally
https://github.com/coronasafe/care_fe/issues/2422#issuecomment-1133250416
Shared a comment on an issue in coronasafe/care_fe on May 20, 2022 at 7:28:26 PM
Regression from coronasafe/care#715
https://github.com/coronasafe/care_fe/issues/2452#issuecomment-1133248795
Shared a comment on a pull request in coronasafe/care on May 20, 2022 at 8:33:52 AM
Requires #724 for static files
https://github.com/coronasafe/care/pull/768#issuecomment-1132633036
Shared a comment on an issue in coronasafe/care_fe on May 19, 2022 at 11:07:47 AM
related to coronasafe/care#725
https://github.com/coronasafe/care_fe/issues/2437#issuecomment-1131552907
Shared a comment on an issue in coronasafe/care_fe on May 19, 2022 at 11:05:38 AM
This issue can be combined with coronasafe/care_fe#2285
https://github.com/coronasafe/care_fe/issues/2435#issuecomment-1131551086
Shared a comment on a pull request in coronasafe/care on May 18, 2022 at 10:59:04 AM
@Marmik2003 you changes from other PR are being mixed up in ths one To fix the problem: Close this PR create a new branch from fd027983dcdf6ce436b5499183a57ac24dbedd98 and cherry-pick the changes on that branch.
https://github.com/coronasafe/care/pull/735#issuecomment-1129866720
Shared a comment on a pull request in coronasafe/care_fe on May 18, 2022 at 5:41:10 AM
@cp-Coder tailwind SS/custom-forms needs to be migrated to tailwindcss/forms https://tailwindcss.com/blog/tailwindcss-v2#utility-friendly-form-styles
https://github.com/coronasafe/care_fe/pull/2402#issuecomment-1129589643
Shared a comment on an issue in coronasafe/care on May 17, 2022 at 12:25:52 PM
alteady pointed out coronasafe/care_fe#2298
https://github.com/coronasafe/care/issues/750#issuecomment-1128805150
Shared a comment on an issue in coronasafe/care on May 15, 2022 at 11:47:39 AM
@siddnikh we are still on django django 2.2
https://github.com/coronasafe/care/issues/745#issuecomment-1126919975
Shared a comment on a pull request in coronasafe/care_fe on May 12, 2022 at 10:35:51 AM
@skks1212 can you try making the time and rhythm column a bit narrower because they don't need that much space.
https://github.com/coronasafe/care_fe/pull/2313#issuecomment-1124832537
worked on a secret project :grin:
created <https://github.com/coronasafe/care/pull/777|#777> to handle an integrity error
Created a PR <https://github.com/coronasafe/care/pull/772> Tracked the error in <https://github.com/coronasafe/care_fe/issues/2452> Tried to reproduce <https://github.com/coronasafe/care_fe/issues/2422>
Created a PR on care_fe and picked up an issue on the backend <https://github.com/coronasafe/care_fe/pull/2391>
EOD for 16th created issues for outdated dependencies in care_fe tried fixing some of them
usage: `./contrib.py sainak 05-15` result: ```Contributions by sainak between 2022-05-09..2022-05-15: Issues: Show the names of the creator in user api <https://github.com/coronasafe/care/issues/744> Pulls: added a new created_by field in user model <https://github.com/coronasafe/care/pull/746> fix: Use latest node LTS everywhere <https://github.com/coronasafe/care_fe/pull/2358> feat: added devcontainer config for vscode <https://github.com/coronasafe/care_fe/pull/2311> feat: use rescript compiler instead of bsb <https://github.com/coronasafe/care_fe/pull/2306>```
created a minor PR on care_fe and another PR on care to add a new field for saving the creator of a user
I need to prepare for exams so not much for today just some minor stuff here and there.
Today I worked on Care Fe to add devcontainer config and started fixing test cases in care
Explored care repo and looked for issue that I can work on also discussed some possible upgrades for care container
Worked on <https://github.com/coronasafe/care/pull/811|https://github.com/coronasafe/care/pull/811> Cleared few issues from sentry Reviewed some PRs
Nothing for today, worked on improving the performance of my laptop's kernel
Trying to debug a wierd issue with django
Created an epic to track dependency updates in care And made a pr <https://github.com/coronasafe/care/pull/836|https://github.com/coronasafe/care/pull/836>
Fixed issues with existing PR to add documentation build and deploy pipeline
<@U02TB8XHZD1> has joined the channel
EOD 21/06/2023 <!subteam^S035L5S02RF|@care_engineering> How did the day go? • Fixed issues on the staging deployment of django 4.2 • updated boto3 to the latest version along with adding static type checking for boto3 in vscode • Had an onboarding call with Shahid along with <@U02TDGQQPMJ> and <@U02T4HFA07Q> What's next? • Stabilize django 4.2 changes and get them ready for production
EOD 28/06/2023 <!subteam^S035L5S02RF|@care_engineering> How did the day go? • polished the logo for pwa with the standard specification • worked on the RBAC draft • created cloudflare tunnels for testing the pwa install, caching always finds a way to mess with your flow EOD 30/06/2023 How did the day go? • Debugged some testing issues • Finished the RBAC draft • Had 1:1 sessions with the c4gt members What's next? • get the draft reviewed by vignesh and start working on the implementation
EOD 01/07/2023 <!subteam^S035L5S02RF|@care_engineering> How did the day go? • Fixed a dependency issue with fihr.resources <https://github.com/coronasafe/care/pull/1418|#1418> • Removed unexpected text from discharge summary <https://github.com/coronasafe/care/pull/1420|#1420> • Traced recent 502 errors on login endpoint <https://github.com/coronasafe/care/issues/1421|#1421> • reviewed some backend PRs *What's next?* • Get my internet fixed
EOD 03/07/2023 <!subteam^S035L5S02RF|@care_engineering> How did the day go? • Fixed an issue where the exports were malfunctioning and returning incomplete data <https://github.com/coronasafe/care_fe/issues/5823|#5823> • After confirming with Mathew that the hba1c field inside consultation is not being used anywhere, updated my pull request to remove the field completely<https://github.com/coronasafe/care/pull/1309| #1309> • was having issues with my internet so I got a fiber connection today, things should be better now • looked for issues to work on Whats next? • start working on bed types v2
EOD 04/07/2023 <!subteam^S035L5S02RF|@care_engineering> How did the day go? • Worked on Bed types v2 • Debugged issues with cypress failing due the backend • Reviewed some Backend PRs Whats up next? • finish working on a basic implementation of bed types v2 • start working on new RBAC
EOD 05/07/2023 <!subteam^S035L5S02RF|@care_engineering> How did the day go? • Reviewed PRs #1072 #1427 #1430 #1106 #1432 #1167 #1063 #1058 • Updated PRs #1329 #1370 #1431 • Created PRs #1437 • Created Issue #1434 • Profiling and debugging django Whats up next? • optimise the backend • work on new bed types
EOD 06/07/2023 How did the day go? • worked on new bed types • worked on adding full text search to medibase table along with rithvik • did some profiling of backend endpoints • reviewed some C4GT PRs Whats up next? • work on new bed types
*EOD 10/07/2023* <!subteam^S035L5S02RF|@care_engineering> How did the day go? • finished working on bed type v2 • created tests for bed type v2 *EOD 11/07/2023* *How did the day go?* • fixed MSE authentication on middleware • encountered another issue with celery workers • had a deep dive of care infrastructure of production and staging deployments from mathew Whats up next? • add audit table for HumanResource allocation
EOD 13/07/2023 <!subteam^S035L5S02RF|@care_engineering> how did the day go ? • added tests and migration to backfill user facility allocation tracking • removed unused feed components from fe • worked on making openID auth (care->muddleware) configurable what's next • remove unessary token dependencies (currently 4 in middleware and 3 in care)
EOD 17/07/2023 <!subteam^S035L5S02RF|@care_engineering> How did the day go ? • fixed edge cases with my old PR • reviewed some PRs What’s next ? • clear out old PRs • find issues to work on
EOD 18/07/2023 <!subteam^S035L5S02RF|@care_engineering> How did the day go ? • Converted zoomdrive to typescript EOD 19/07/2023 How did the day go ? • Fixed issues on the backend #1459 #1461 #1464 • Reviewed some PRs and sent them to be merged What's next? • clear up more PRs and sentry issues
EOD 20/07/2023 <!subteam^S035L5S02RF|@care_engineering> How did the day go? • Worked on refactoring notification handler • Reviewed and updated backend backend PRs to get them to mergeable state • Reviewed ABDM Whats up next? • Work on notification handler • Fix more issues reported by sentry