Personal Learning Dashboard (Beta)

ShivamJhaa

Shivam Jha

Contributor

Graduate attribute
1x
Graduate attribute
1x
Graduate attribute
Graduate attribute
1x
Graduate attribute
1x
Graduate attribute
1x
Graduate attribute
1x
Graduate attribute
1x
Graduate attribute
1x
Graduate attribute
Graduate attribute
1x
Graduate attribute
Graduate attribute
Graduate attribute

Graduate Attributes

Learn More

Individual Skills

Graduate attribute1x

Creative thinking

Graduate attribute1x

Problem Solving

Graduate attribute

Practical/Professional Skills

Team Skills

Graduate attribute1x

Communication Skills

Graduate attribute1x

Collaboration

Graduate attribute1x

Community Engagement

Graduate attribute1x

Leadership

Graduate attribute1x

Learn How to Learn

Graduate attribute1x

Skills to apply digital & technology solutions

Graduate attribute

Critical Thinking

Advanced Skills

Graduate attribute1x

Autonomy And Responsibility

Graduate attribute

Empathy

Graduate attribute

Value inculcation

Graduate attribute

Multicultural Competence

Cultural Skills

Short Bio

Still waiting for this

Learning Activity

Highlights

Pull Request
6

0 in last 7 days

Reviews
4

0 in last 7 days

Feed
0

0 in last 7 days

Avg. PR Turnaround Time

N/A

Yet to make contributions!

Contributions

  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6135
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6135
  • Shared a comment on a pull request in coronasafe/care_fe on September 8, 2023 at 4:10:14 PM

    @khavinshankar There was some issue in submitting the consultation, that's why the test was failing. Please take a look, now test are passing.

    https://github.com/coronasafe/care_fe/pull/6135#issuecomment-1711913044

  • closed an issue on coronasafe/care_fe
    Add test for facility.cy.tsSeptember 7, 2023 at 1:24:34 PM
  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6106
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6106
  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5870
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5870
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • reviewed a pull request on coronasafe/care_fe
  • Shared a comment on a pull request in coronasafe/care_fe on August 25, 2023 at 6:58:47 AM

    @nihal467 I have fixed the merge conflicts, please take a look Thanks!

    https://github.com/coronasafe/care_fe/pull/5870#issuecomment-1692865828

  • opened a pull request on coronasafe/care_fe
    Added test of patient_crud.cy.tsAugust 22, 2023 at 5:28:36 PM
  • opened an issue on coronasafe/care_fe
    Add test for resources.cy.tsAugust 19, 2023 at 5:42:18 AM
  • opened an issue on coronasafe/care_fe
    Add test for facility.cy.tsAugust 19, 2023 at 5:38:59 AM
  • opened a pull request on coronasafe/care_fe
    Added test for resource.cy.tsAugust 19, 2023 at 4:40:13 AM
  • opened a pull request on coronasafe/care_fe
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5876
  • opened a pull request on coronasafe/care_fe
  • opened an issue on coronasafe/care_fe
    Add test for patient_crud.cy.tsJuly 12, 2023 at 10:33:53 AM
  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5841
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5841
  • Shared a comment on a pull request in coronasafe/care_fe on July 9, 2023 at 5:59:25 PM

    > @ShivamJhaa > > * As per our call, we discussed to verify mandatory field error message, so initially click on the submit button and verify all mandatory field are having error message reflected to the existing field > * you only created single asset using create asset button, missed out the functionality check of create and add more button, include that in the test and add multiple asset through it > * verify the asset is created successfully in the asset tab, by searching with it name ( Only check during single asset creation time ) * Added error message verification test * Added test for the functionality of create and add more and also added check to verify whether the asset is present or not Please take a look

    https://github.com/coronasafe/care_fe/pull/5841#issuecomment-1627784895

  • Shared a comment on a pull request in coronasafe/care_fe on July 5, 2023 at 8:19:19 PM

    @nihal467 Please review the PR, thanks!

    https://github.com/coronasafe/care_fe/pull/5841#issuecomment-1622438102

  • opened a pull request on coronasafe/care_fe
  • opened an issue on coronasafe/care_fe
    Add test for assets.cy.tsJuly 4, 2023 at 11:57:35 AM
  • opened an issue on coronasafe/care_fe
    Cypress Test for All Major Functionalities of CAREJuly 4, 2023 at 11:56:04 AM
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5713
  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5632
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5632
  • Shared a comment on a pull request in coronasafe/care_fe on June 12, 2023 at 5:48:59 PM

    > @ShivamJhaa before pushing the PR, make sure that you revert the changes you are making in vite.config.ts to run your own backend Sure, will keep this in mind, thanks!

    https://github.com/coronasafe/care_fe/pull/5632#issuecomment-1587799391

  • reviewed a pull request on coronasafe/care_fe
    Added cypress test for asset creationJune 12, 2023 at 5:46:41 PM
  • Shared a comment on a pull request in coronasafe/care_fe on June 7, 2023 at 11:47:40 AM

    @khavinshankar @nihal467 The PR seems to be stale, can I take up the task?

    https://github.com/coronasafe/care_fe/pull/4134#issuecomment-1580630444

  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5549
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5549
  • Shared a comment on a pull request in coronasafe/care_fe on June 1, 2023 at 11:42:28 AM

    @khavinshankar Now the test is passing on Cl as well.

    https://github.com/coronasafe/care_fe/pull/5549#issuecomment-1571894740

  • Shared a comment on a pull request in coronasafe/care_fe on June 1, 2023 at 10:41:31 AM

    @khavinshankar Its strange that it passes successfully locally but fails on CI: ![Screenshot from 2023-06-01 16-08-02](https://github.com/coronasafe/care_fe/assets/86483059/de9a24bd-ac8d-4481-91de-a529f1262c9e) Can you please re-run the test?

    https://github.com/coronasafe/care_fe/pull/5549#issuecomment-1571801710

  • Shared a comment on a pull request in coronasafe/care_fe on May 31, 2023 at 7:39:50 AM

    @rithviknishad @nihal467 Can you please review the PR? Thanks!

    https://github.com/coronasafe/care_fe/pull/5549#issuecomment-1569658946

  • reviewed a pull request on coronasafe/care_fe
  • Shared a comment on an issue in coronasafe/care_fe on May 30, 2023 at 12:24:18 PM

    I was able to fix the error, by using ID as a selector for the input box (this also removed the need of `force: true`). I have pushed all the changes.

    https://github.com/coronasafe/care_fe/issues/4233#issuecomment-1568342217

  • Shared a comment on an issue in coronasafe/care_fe on May 30, 2023 at 10:44:03 AM

    Removing `force: true` throws this error: ![Screenshot from 2023-05-30 16-13-15](https://github.com/coronasafe/care_fe/assets/86483059/747efe52-ce39-4ce9-9e77-6e4c2a4a1e8d)

    https://github.com/coronasafe/care_fe/issues/4233#issuecomment-1568208568

  • Shared a comment on an issue in coronasafe/care_fe on May 30, 2023 at 10:30:43 AM

    > @ShivamJhaa > > ![image](https://user-images.githubusercontent.com/57055998/241570190-ec4fcef2-ff61-4606-95db-1ed4504e2f61.png) > > * keep the phone number constant as "9999999999" for now. > * Once you enter the phone number, a pop-up will appear, similar to the screenshot above. You need to click on "I confirm that the suspect/patient I want to create is not on the list" and continue filling in the remaining fields. > * try solving this, because you are using " force: true " in the code to input data to the phone number field, which disables all pop-ups and related validations. so solve without using force:true > * For now, only include the mandatory fields. Once you figure it out, you can input data into the optional fields. This solution will work for the phone number field but we will still have an invalid emergency phone number and this will not allow us to create a new patient.

    https://github.com/coronasafe/care_fe/issues/4233#issuecomment-1568192544

  • Shared a comment on an issue in coronasafe/care_fe on May 27, 2023 at 11:48:18 AM

    And due to this only we are not able to get the URL of the patient created which is needed for the next test cases and hence they are failing.

    https://github.com/coronasafe/care_fe/issues/4233#issuecomment-1565368909

  • Shared a comment on an issue in coronasafe/care_fe on May 27, 2023 at 11:45:48 AM

    @nihal467 Actually I have not added any check in the end to verify that the Patient is created, that's why it is showing passed. If you will pause the test at the end you will see that after clicking on the Add Patient button instead of creating the patient it throws error.

    https://github.com/coronasafe/care_fe/issues/4233#issuecomment-1565367510

  • Shared a comment on an issue in coronasafe/care_fe on May 27, 2023 at 7:51:51 AM

    > @ShivamJhaa try all these formats > > * "919994565656" > * "+919994565656" > * "+91 9994565656" > * "+91 99945-65656" > one of these might work. > > @GokulramGHV can you help @ShivamJhaa out Strange none of them worked, still giving the same error. ![Screenshot from 2023-05-27 13-21-03](https://github.com/coronasafe/care_fe/assets/86483059/0915f852-7709-4503-967e-d03324941ef4)

    https://github.com/coronasafe/care_fe/issues/4233#issuecomment-1565259054

  • opened a pull request on coronasafe/care_fe
  • Shared a comment on an issue in coronasafe/care_fe on May 26, 2023 at 2:43:01 PM

    @khavinshankar , @nihal467 While writing the test for new patient creation, I encountered the following error. Even after typing the phone number when we click on Add Patient button, we get the following error: ![Screenshot from 2023-05-26 19-55-23](https://github.com/coronasafe/care_fe/assets/86483059/12b4159f-724c-4c74-a49e-9c300baa98ea) We can clearly see that the phone number typed is a valid one. I tried debugging this and logged the input value we are getting from the form. The value we get is something like: "919994565656" and when we try to take out the phone number from this input we get an undefined value. One strange is if I edit anything when the test is paused and again try to submit the form, this time we get "+91 956465656" (notice the + sign and gap) and we are able to successfully retrieve the phone number from it using `parsePhoneNumberFromString` method. Check out the screenshot for more info: ![Screenshot from 2023-05-26 20-02-09](https://github.com/coronasafe/care_fe/assets/86483059/29a07702-a061-4ef2-866b-c698d77810f1) The first set of console values are when the phone numbers are typed using cypress, the next one is when I tried editing one of them and re submit the form. Can anyone of you please take a look at this. Thanks!

    https://github.com/coronasafe/care_fe/issues/4233#issuecomment-1564500770

  • Shared a comment on an issue in coronasafe/care_fe on May 24, 2023 at 7:33:39 AM

    @khavinshankar I would like to work on this issue, can you please assign me this task? Thank You!

    https://github.com/coronasafe/care_fe/issues/4233#issuecomment-1560599130

  • Shared a comment on an issue in coronasafe/care_fe on May 22, 2023 at 1:47:38 PM

    @nihal467 All the unclaimed tasks depend on the open PRs like adding, and editing consultation can only be done after creating patients, and configuration of the assets page can be done after creating an asset. So can I take those tasks which are already assigned (creating an asset) as the PR seems stale now?

    https://github.com/coronasafe/care_fe/issues/4133#issuecomment-1557253346

More to come in the coming days...!