ohc.networkContributors

Personal Learning Dashboard (Beta)

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

MonWedFriDecJanFebMarAprMayJunJulAugSepOctNovDec
36 contributions in the last year
LessMore

Highlights

Pull Request
7
Reviews
0
Feed
0
Avg. PR Turnaround Time
21 hours 10 seconds

Week Summary

Pull Requests

Opened
0
Reviewed
0
Merged
1
Stale
0

Activity

EOD Updates
0
Comments
1

Contributions

  • merged a pull request on coronasafe/care_fe with a turnaround time of 21 hours 10 seconds
    https://github.com/coronasafe/care_fe/pull/6806
  • Shared a comment on a pull request in coronasafe/care_fe on December 12, 2023 at 11:51:12 AM

    @nihal46 > once i select it, i can unselect the option, by clicking on the button Do you mean the button should toggle the `clone_last` variable? In that case I think it would have been better to keep it as checkbox, as done in my previous commit. (https://github.com/coronasafe/care_fe/pull/6659/commits/ad57c3c50bcbd9fd95d843a6f99e137e7f2364ba) Button is going to be confusing for users. The button has text "Copy values from previous log". If we use this to toggle, we need to change the button text also. What should the text be when button is used to toggle `clone_last` to false? PS: Still thinking checkbox is correct control here. In case we still want to move forward with button - let me know what button text should be in both states. Thanks.

    https://github.com/coronasafe/care_fe/pull/6659#issuecomment-1851888134

  • opened a pull request on coronasafe/care_fe
  • Shared a comment on an issue in coronasafe/care_fe on December 7, 2023 at 7:52:32 AM

    I have a fix for this. @rithviknishad can you please assign this to me so I can send PR. Thanks.

    https://github.com/coronasafe/care_fe/issues/6805#issuecomment-1844837174

  • opened an issue on coronasafe/care_fe
    occupancy indicator in facility card does not resize itself when font size is changed in browserDecember 7, 2023 at 7:50:19 AM
  • Shared a comment on a pull request in coronasafe/care_fe on December 5, 2023 at 4:37:45 AM

    @nihal467 The suggestion to use button was for when we had made changes to display the values of previous log in this form. Idea was by clicking the button, form would get filled with previous log values. After you suggested to revert all changes and just add the description of what will happen I have added just that message "Field values will be copied from previous log update". The form no longer displays what values are being copied. The copying is going to happen entirely on backend.

    https://github.com/coronasafe/care_fe/pull/6659#issuecomment-1839993091

  • Shared a comment on a pull request in coronasafe/care_fe on November 28, 2023 at 6:01:46 AM

    Minor change based on Bodhi' feedback. Made border rounded and used softer colors. ![image](https://github.com/coronasafe/care_fe/assets/56870381/60bb9f57-708a-4205-b17e-4a2df892602c)

    https://github.com/coronasafe/care_fe/pull/6659#issuecomment-1829171740

  • pr_collaborated
    Production Release Nov Week 4
    November 28, 2023 at 4:25:07 AM
  • Shared a comment on an issue in coronasafe/care_fe on November 23, 2023 at 5:58:38 PM

    I'd like to take this up. Can you assign me @rithviknishad

    https://github.com/coronasafe/care_fe/issues/6705#issuecomment-1824783365

  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6567
  • pr_collaborated
    Improve rounding of the preset buttons. Fixes #6555
    November 21, 2023 at 11:40:21 AM
  • pr_collaborated
    Production Release Mid November
    November 21, 2023 at 6:46:36 AM
  • opened a pull request on coronasafe/care_fe
    Show fields when copying from previous logNovember 17, 2023 at 6:07:08 AM
  • Shared a comment on an issue in coronasafe/care_fe on November 15, 2023 at 10:43:52 AM

    I'll like to work on this.

    https://github.com/coronasafe/care_fe/issues/6617#issuecomment-1812231475

  • Shared a comment on an issue in coronasafe/care_fe on November 15, 2023 at 10:13:07 AM

    @aparnacoronasafe what should happen when user unchecks the checkbox? will it be more appropriate to turn this checkbox into a button?

    https://github.com/coronasafe/care_fe/issues/6602#issuecomment-1812160725

  • opened a pull request on coronasafe/care_fe
  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6515
  • pr_collaborated
    Make header in patient dashboard more responsive. Fixes #6488
    October 31, 2023 at 11:21:08 AM
  • pr_collaborated
    Production Release; Nov Week 1
    October 29, 2023 at 5:33:47 PM
  • opened a pull request on coronasafe/care_fe
  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6485
  • Shared a comment on an issue in coronasafe/care_fe on October 25, 2023 at 6:28:01 AM

    assign me. thanks

    https://github.com/coronasafe/care_fe/issues/6488#issuecomment-1778596305

  • opened a pull request on coronasafe/care_fe
    Configure eslint for mdx files. Closes #3760October 24, 2023 at 10:28:39 AM
  • Shared a comment on an issue in coronasafe/care_fe on October 19, 2023 at 5:53:09 AM

    Should this be close now because we are going to use WebRTC instead?

    https://github.com/coronasafe/care_fe/issues/6321#issuecomment-1770105152

  • Shared a comment on an issue in coronasafe/care_fe on October 18, 2023 at 8:16:45 AM

    Can assign this to me.

    https://github.com/coronasafe/care_fe/issues/3760#issuecomment-1767927774

  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6415
  • pr_collaborated
    Optimize lodash. Fixes #6006
    October 18, 2023 at 4:53:02 AM
  • pr_collaborated
    Production Release; Oct Week 3
    October 15, 2023 at 9:40:20 AM
  • Shared a comment on a pull request in coronasafe/care_fe on October 12, 2023 at 9:10:56 AM

    Hi @bodhish, Working on your suggestion to use css text-transform instead of `startCase` and `camelCase`. From what I'm seeing I think it won't be able to replace those methods . Correct me if wrong > Is a keyword that converts the first letter of each word to uppercase. Other characters remain unchanged (they retain their original case as written in the element's text). https://developer.mozilla.org/en-US/docs/Web/CSS/text-transform `sampleDetails.status` can have value "COMPLETED". We need "Completed". `capitalize` class won't do that. Similar issue for `sampleDetails.doctor_name` and maybe in other places. https://play.tailwindcss.com/wR1YZEkNsO

    https://github.com/coronasafe/care_fe/pull/6415#issuecomment-1759229691

  • merged a pull request on coronasafe/care_fe
    https://github.com/coronasafe/care_fe/pull/6424
  • Shared a comment on an issue in coronasafe/care_fe on October 10, 2023 at 3:07:41 PM

    I can take this issue.

    https://github.com/coronasafe/care_fe/issues/6321#issuecomment-1755650627

  • opened a pull request on coronasafe/care_fe
    Using the treeshakable api for echartsOctober 9, 2023 at 10:50:26 AM
  • Shared a comment on a pull request in coronasafe/care_fe on October 9, 2023 at 10:31:09 AM

    > @Omkar76 wasn't the issue about removing and using custom utility functions instead? > > This PR is just replacing with another package and updating the imports and doesn't make a noticeable difference in the final bundle size. > > We could achieve a much lower bundle size if we built the utility functions within care right? > > cc: @sainak I believe the size reduction is significant (547 -> 122). We can keep this PR on hold until @sainak has implemented the utilities to replace lodash. If that reduces size significantly more than this we can merge that instead.

    https://github.com/coronasafe/care_fe/pull/6415#issuecomment-1752751992

  • Shared a comment on a pull request in coronasafe/care_fe on October 7, 2023 at 7:44:38 AM

    cc: @sainak

    https://github.com/coronasafe/care_fe/pull/6415#issuecomment-1751639859

  • opened a pull request on coronasafe/care_fe
    Optimize lodash. Fixes #6006October 7, 2023 at 7:41:55 AM
  • Shared a comment on an issue in coronasafe/care_fe on September 30, 2023 at 8:41:02 AM

    You can assign this issue to me.

    https://github.com/coronasafe/care_fe/issues/6373#issuecomment-1741716463

More to come in the coming days...!