ohc.networkContributors

Personal Learning Dashboard (Beta)

sachdevavaibhav
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
5

0 in last 7 days

Feed
0

0 in last 7 days

Avg. PR Turnaround Time

N/A

Yet to make contributions!

Contributions

  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6068
  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5730
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5730
  • reviewed a pull request on coronasafe/care_fe

  • Shared a comment on a pull request in coronasafe/care_fe on August 8, 2023 at 11:39:28 AM

    > @sachdevavaibhav can you fix the merge conflict and get it ready for testing, we need to get it merge ASAP solved merge conflict. Though I have one question why are we using moment.js and day.js both. Don't they serve the same purpose. I have kept the code coming from develop which is using dayjs. @nihal467 @Ashesh3

    https://github.com/coronasafe/care_fe/pull/5730#issuecomment-1669451311

  • Shared a comment on a pull request in coronasafe/care_fe on August 3, 2023 at 3:47:39 AM

    > @sachdevavaibhav what is the status on this PR solved merge conflict. pushed necessary changes. Kindly review it. @Ashesh3

    https://github.com/coronasafe/care_fe/pull/5730#issuecomment-1663249774

  • Shared a comment on a pull request in coronasafe/care_fe on July 21, 2023 at 6:46:10 AM

    @rithviknishad Changes are merged in the backend. You may merge this as well.

    https://github.com/coronasafe/care_fe/pull/5730#issuecomment-1645064549

  • merged a pull request on coronasafe/care
    https://github.com/coronasafe/care/pull/1430
  • collaborated on a pull request on coronasafe/care
    https://github.com/coronasafe/care/pull/1430

  • Shared a comment on a pull request in coronasafe/care on July 10, 2023 at 6:09:48 AM

    > > > @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 > > @sainak I have written a few test and found that its also accepting integers (may be its converting it to string values). Kindly review the test and should I also add some validation to the input? @sainak

    https://github.com/coronasafe/care/pull/1430#issuecomment-1628301526

  • Shared a comment on a pull request in coronasafe/care on July 7, 2023 at 11:46:23 AM

    > > @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 @sainak I have written a few test and found that its also accepting integers (may be its converting it to string values). Kindly review the test and should I also add some validation to the input?

    https://github.com/coronasafe/care/pull/1430#issuecomment-1625292647

  • Shared a comment on a pull request in coronasafe/care on July 6, 2023 at 6:36:05 PM

    > > @sachdevavaibhav can you add some tests to validate whether this field is accepting correct inputs > > ya sure! @sainak I have a couple of questions: - referred_to_external should accept only strings/null values right? And not integer etc. - Should I create separate branch for tests or push them here only?

    https://github.com/coronasafe/care/pull/1430#issuecomment-1624144636

  • Shared a comment on a pull request in coronasafe/care on July 5, 2023 at 8:30:07 AM

    > @sachdevavaibhav can you add some tests to validate whether this field is accepting correct inputs ya sure!

    https://github.com/coronasafe/care/pull/1430#issuecomment-1621284494

  • reviewed a pull request on coronasafe/care

  • Shared a comment on a pull request in coronasafe/care on July 4, 2023 at 5:35:52 PM

    @rithviknishad Done with formatting.

    https://github.com/coronasafe/care/pull/1430#issuecomment-1620571256

  • Shared a comment on a pull request in coronasafe/care_fe on July 4, 2023 at 4:14:55 PM

    > @sachdevavaibhav yes! Go ahead! Done! Kindly review it.

    https://github.com/coronasafe/care_fe/pull/5730#issuecomment-1620501263

  • Shared a comment on a pull request in coronasafe/care on July 4, 2023 at 4:12:42 PM

    @rithviknishad Kindly review it.

    https://github.com/coronasafe/care/pull/1430#issuecomment-1620498998

  • opened a pull request on coronasafe/care
  • opened an issue on coronasafe/care
    PatientConsultationDischargeSerializer is not having referred_to_external field.July 4, 2023 at 3:44:24 PM

  • Shared a comment on a pull request in coronasafe/care_fe on July 4, 2023 at 10:28:08 AM

    @rithviknishad Should I create issue and link a PR for the backend?

    https://github.com/coronasafe/care_fe/pull/5730#issuecomment-1619982541

  • Shared a comment on a pull request in coronasafe/care_fe on July 1, 2023 at 6:06:45 PM

    @rithviknishad Upon my analysis I found the value is not being stored in the database. Its storing a null value for it. I found this by setting up care backend locally. It is storing other fields like discharge_reason etc correctly but storing null for referred_to_external.

    https://github.com/coronasafe/care_fe/pull/5730#issuecomment-1616024879

  • Shared a comment on a pull request in coronasafe/care_fe on June 24, 2023 at 5:15:03 PM

    @rithviknishad @nihal467

    https://github.com/coronasafe/care_fe/pull/5730#issuecomment-1605655544

  • Shared a comment on a pull request in coronasafe/care_fe on June 20, 2023 at 9:34:22 AM

    @rithviknishad Kindly review it. The field can search from the options but it is not able to add values which are not present. Please suggest how can I solve it?

    https://github.com/coronasafe/care_fe/pull/5730#issuecomment-1598442215

  • opened a pull request on coronasafe/care_fe

  • Shared a comment on an issue in coronasafe/care_fe on June 17, 2023 at 9:03:24 AM

    okay

    https://github.com/coronasafe/care_fe/issues/5407#issuecomment-1595679850

  • Shared a comment on an issue in coronasafe/care_fe on June 17, 2023 at 9:00:41 AM

    @rithviknishad Is there any defined component for this issue already. We need a select menu with free text. Should I make modifications with `SelectMenuV2`?

    https://github.com/coronasafe/care_fe/issues/5407#issuecomment-1595679306

  • Shared a comment on an issue in coronasafe/care_fe on June 9, 2023 at 4:07:51 PM

    @rithviknishad If no one is working on it you can assign it to me.

    https://github.com/coronasafe/care_fe/issues/5407#issuecomment-1584823195

  • Shared a comment on an issue in coronasafe/care_fe on June 9, 2023 at 4:03:57 PM

    @rithviknishad I can work on it.

    https://github.com/coronasafe/care_fe/issues/5633#issuecomment-1584818181

  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5570
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5570

  • Shared a comment on an issue in coronasafe/care_fe on June 4, 2023 at 2:42:16 PM

    @print-Sathvik Thanks

    https://github.com/coronasafe/care_fe/issues/4941#issuecomment-1575594749

  • Shared a comment on an issue in coronasafe/care_fe on June 3, 2023 at 12:15:19 PM

    @rithviknishad Can I work on this issue? If yes, kindly assign it to me.

    https://github.com/coronasafe/care_fe/issues/4941#issuecomment-1574904707

  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5588
  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5553
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5553

  • Shared a comment on a pull request in coronasafe/care_fe on May 30, 2023 at 4:49:59 PM

    ![Screenshot 2023-05-30 221859](https://github.com/coronasafe/care_fe/assets/72242181/b57ad3d6-bf4d-48c0-b637-08db061421ae) @rithviknishad Kindly review the changes.

    https://github.com/coronasafe/care_fe/pull/5570#issuecomment-1568759108

  • reviewed a pull request on coronasafe/care_fe

  • Shared a comment on a pull request in coronasafe/care_fe on May 30, 2023 at 4:33:53 PM

    @nihal467 Solved the merge conflict. Kindly review it.

    https://github.com/coronasafe/care_fe/pull/5553#issuecomment-1568739779

  • Shared a comment on a pull request in coronasafe/care_fe on May 30, 2023 at 3:45:01 PM

    @nihal467 Ya sure!

    https://github.com/coronasafe/care_fe/pull/5553#issuecomment-1568668119

  • Shared a comment on an issue in coronasafe/care_fe on May 30, 2023 at 1:32:28 PM

    @rithviknishad Kindly review the PR.

    https://github.com/coronasafe/care_fe/issues/4946#issuecomment-1568444552

  • opened a pull request on coronasafe/care_fe
  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5534
  • collaborated on a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/5534

  • Shared a comment on an issue in coronasafe/care_fe on May 29, 2023 at 3:05:06 AM

    @rithviknishad I can work on this issue. Kindly assign it to me.

    https://github.com/coronasafe/care_fe/issues/4946#issuecomment-1566432047

  • reviewed a pull request on coronasafe/care_fe
    Sachdevavaibhav/issue#4945May 28, 2023 at 5:46:29 PM

  • Shared a comment on a pull request in coronasafe/care_fe on May 28, 2023 at 4:04:51 PM

    @rithviknishad I have made the changes. Kindly review it.

    https://github.com/coronasafe/care_fe/pull/5553#issuecomment-1566184979

  • Shared a comment on an issue in coronasafe/care_fe on May 28, 2023 at 10:53:33 AM

    @rithviknishad Please review the PR #5553

    https://github.com/coronasafe/care_fe/issues/4945#issuecomment-1566084556

  • opened a pull request on coronasafe/care_fe
    Sachdevavaibhav/issue#4945May 28, 2023 at 10:52:11 AM

  • Shared a comment on a pull request in coronasafe/care_fe on May 28, 2023 at 10:45:56 AM

    @rithviknishad I have reverted changes in the previous branch and force pushed it (since I merged develop into it). Closing this PR and creating new one.

    https://github.com/coronasafe/care_fe/pull/5551#issuecomment-1566081834

  • Shared a comment on a pull request in coronasafe/care_fe on May 28, 2023 at 8:36:39 AM

    @rithviknishad Yes, I didn't checkout to develop. I pulled develop into the previous branch and then created a new branch. Sorry for the mess. Any suggestions on how I can fix it?

    https://github.com/coronasafe/care_fe/pull/5551#issuecomment-1566002388

  • Shared a comment on a pull request in coronasafe/care_fe on May 27, 2023 at 9:09:53 PM

    @rithviknishad Could you please check if I am moving in the right direction? Also did I mess up with the branches?

    https://github.com/coronasafe/care_fe/pull/5551#issuecomment-1565684600

  • opened a pull request on coronasafe/care_fe
    Sachdevavaibhav/#4945May 27, 2023 at 8:52:38 PM

  • Shared a comment on an issue in coronasafe/care_fe on May 27, 2023 at 3:54:28 PM

    @rithviknishad Sure! I am working on it.

    https://github.com/coronasafe/care_fe/issues/4945#issuecomment-1565542948

  • Shared a comment on an issue in coronasafe/care_fe on May 27, 2023 at 3:48:56 PM

    @khavinshankar It looks like this issue has already been fixed PR #5062. I suggest you to close this and the other issues solved by the mentioned PR.

    https://github.com/coronasafe/care_fe/issues/4945#issuecomment-1565536681

  • Shared a comment on an issue in coronasafe/care_fe on May 26, 2023 at 8:35:37 PM

    I can work on this issue. Please assign it to me.

    https://github.com/coronasafe/care_fe/issues/4945#issuecomment-1564917482

  • Shared a comment on a pull request in coronasafe/care_fe on May 26, 2023 at 9:14:51 AM

    @rithviknishad Thankyou for the information. Will look into solving other issues.

    https://github.com/coronasafe/care_fe/pull/5534#issuecomment-1564077123

  • Shared a comment on a pull request in coronasafe/care_fe on May 26, 2023 at 8:38:40 AM

    @rithviknishad Could you please guide me how can I contribute to testing?

    https://github.com/coronasafe/care_fe/pull/5534#issuecomment-1564020439

  • reviewed a pull request on coronasafe/care_fe

  • Shared a comment on an issue in coronasafe/care_fe on May 26, 2023 at 7:50:32 AM

    @rithviknishad I have created a PR. Kindly review it.

    https://github.com/coronasafe/care_fe/issues/5280#issuecomment-1563959106

  • opened a pull request on coronasafe/care_fe

  • Shared a comment on an issue in coronasafe/care_fe on May 25, 2023 at 10:35:57 PM

    Can I take up this issue?

    https://github.com/coronasafe/care_fe/issues/5530#issuecomment-1563593184

  • opened an issue on coronasafe/care_fe
    Staging API auth credential mentioned in readme not working May 25, 2023 at 10:35:06 PM

  • Shared a comment on an issue in coronasafe/care_fe on May 25, 2023 at 10:15:13 PM

    @rithviknishad This can be fixed using the close prop given by headless UI. Can I take up this issue?

    https://github.com/coronasafe/care_fe/issues/5280#issuecomment-1563574680

  • Shared a comment on an issue in coronasafe/care_fe on May 25, 2023 at 10:11:16 PM

    @rithviknishad This can be solved using the close prop given by headless UI. Could you please assign me this issue?

    https://github.com/coronasafe/care_fe/issues/5280#issuecomment-1563567868

More to come in the coming days...!