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
I build stuff with code.
care/1349
use signals to update patient and bed count of facilitiescare/1309
remove HBA1C field from consultation modelcare_fe/5498
Rename all instances of orgin_facility to origin_facilitycare/1303
Rename all instances of orgin_facility to origin_facilitymock-cns-req/1
fix requests not sending content-type header and minor refactorShared 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