ohc.networkContributors

Personal Learning Dashboard (Beta)

Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute
Graduate attribute

Graduate Attributes

Learn More

Individual Skills

Graduate attribute

Creative thinking

Graduate attribute

Problem Solving

Graduate attribute

Practical/Professional Skills

Team Skills

Graduate attribute

Communication Skills

Graduate attribute

Collaboration

Graduate attribute

Community Engagement

Graduate attribute

Leadership

Graduate attribute

Learn How to Learn

Graduate attribute

Skills to apply digital & technology solutions

Graduate attribute

Critical Thinking

Advanced Skills

Graduate attribute

Autonomy And Responsibility

Graduate attribute

Empathy

Graduate attribute

Value inculcation

Graduate attribute

Multicultural Competence

Cultural Skills

Short Bio

Still waiting for this

Learning Activity

MonWedFriJulAugSepOctNovDecJanFebMarAprMayJun
6 contributions in the last year
LessMore

Highlights

Pull Request
0
Reviews
0
Feed
0

Contributions

  • Shared a comment on an issue in coronasafe/care_fe on November 20, 2023 at 1:57:45 PM

    @rithviknishad any further updates on this?

    https://github.com/coronasafe/care_fe/issues/6612#issuecomment-1819112777

  • Shared a comment on an issue in coronasafe/care_fe on November 20, 2023 at 4:49:44 AM

    I'm happy to make this change on the API side and then come back and fix this. If you're okay with it, please create an issue on the API repo and assign it to me and link this issue there

    https://github.com/coronasafe/care_fe/issues/6612#issuecomment-1818224152

  • Shared a comment on an issue in coronasafe/care_fe on November 20, 2023 at 4:47:16 AM

    @rithviknishad @nihal467 I have taken a deeper look into this, and I can confirm that this needs to be fixed on the API side by adding a new property (ex: configuredMinQuantity) to the inventorysummery response that gives the minimum quantity required for each item. Reason: The invertorysummery API response and the min_quantity API response are both paginated. When the min_quantity list grows bigger there will be a scenario where all the items from invertorysummery might not get their corresponding min_quantity from the min_quantity list, which will lead to displaying the incorrect min quantity on the UI. As API is already returning an is_low indicator, this shouldn't be a big change on the API side, once that is added, we can come back and fix this on the UI

    https://github.com/coronasafe/care_fe/issues/6612#issuecomment-1818222625

  • Shared a comment on an issue in coronasafe/care_fe on November 17, 2023 at 6:53:18 PM

    Can you assign this to me, I can start working on this

    https://github.com/coronasafe/care_fe/issues/6612#issuecomment-1816928958

  • Shared a comment on an issue in coronasafe/care_fe on November 17, 2023 at 4:43:09 AM

    let me know if you are okay with adding this min_quantity call during the page load, I can add this

    https://github.com/coronasafe/care_fe/issues/6612#issuecomment-1815740293

  • Shared a comment on an issue in coronasafe/care_fe on November 17, 2023 at 4:39:26 AM

    Hi @nihal467 @rithviknishad Looked into this a bit, we need to do a getMinInventory call that is being done when the Minimum Quantity Required button is clicked to add this new column. Also, there are two min_quantity fields in the payload, that is not so intutitve, one I assume is a default value and the other is the value set ![image](https://github.com/coronasafe/care_fe/assets/34686116/696c6ea8-eac5-402a-abb4-e74bf71ed39c)

    https://github.com/coronasafe/care_fe/issues/6612#issuecomment-1815737963

More to come in the coming days...!