-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cold Chain: No way for user to acknowledge temperature breach #5180
Comments
Hi @arjunSussol! If you have capacity to look at this, could you do the same as we did for the other issue, and create a proposed solution and estimate for this work please? The best thing to do would be to look at how we do it in the cold chain app - it should work basically the same here. Let me know whether you are available, thank you! :) |
@arjunSussol - happy to talk through anything too! |
@DhanyaHerath @adamdewey today, I have looked at the cold chain module in mobile. |
@arjunSussol - sure that sounds good, ok I'll send an invite in the calendar for the week starting 22nd May and we can have a chat then! |
@arjunSussol & @DhanyaHerath - We are looking to shift to promoting cold chain with Open mSupply for future clients and discourage using the Mobile cold chain module unless there are no alternatives. @Gwalior-84's main client is the only one currently using the Mobile cold chain module. I'm trying to work out whether we need to do this issue at all or if we can keep our head down and free up Arjun and the team to get into a better position for eventually helping out on Open mSupply too. Let me talk to @Gwalior-84 offline and will be in a better position to update on the priority for this. |
@arjunSussol - I spoke to @Gwalior-84 and the decision was that we don't need to do this as we have the expectation that the client will transition to Open mSupply once cold chain is integrated (hopefully by Oct/Nov this year) |
Thanks @adamdewey for the update. I am going to close this issue. |
Ok, these issues have suddenly become relevant again as it looks like F-client will be using Mobile Cold Chain after all. So I'm re-opening this |
Describe the bug
There appears to be no way for a Mobile user to acknowledge a temperature breach captured by the Cold Chain module.
In the Standalone Cold Chain app the user is able to acknowledge the temperature breach:
To reproduce
Steps to reproduce the behaviour:
Expected behaviour
User Stories
User should be able to acknowledge temperature breach by tapping on the triangle icon and selecting 'Acknowledge'
Once acknowledged the corresponding breach sections (hot and cold) should be updated:
Proposed Solution
Need to also update the
acknowledged
field in thetemperature_breach
tableMay also be other tables that need updating - best to check the Standalone Cold Chain app or Desktop to see how they acknowledges temperature breaches
Version and device info
Additional context
This has been reported by a client and so priority has increased accordingly
The text was updated successfully, but these errors were encountered: