Allow user to acknowledge Cold Chain Mobile temperature breaches

[adamdewey]

Is your feature request related to a problem? Please describe.

In the Cold Chain Standalone (SA) app we allow the user to acknowledge temperature breaches. This means that you can use the dashboard to see the overall status of unacknowledged breaches currently in the system.

2021-10-04_11-53-24.mp4

However, with Cold Chain Mobile (M) there doesn't appear to be a way for users to acknowledge the breaches.

Could we add some similar functionality to M?

Describe the solution you'd like

1. User to be able to see unacknowledged breaches and to be able to acknowledge (and dismiss) them.

Perhaps by pressing on the breach statistic panels under the graph?

2. Would need to update the panels to only show numbers for unacknowledged breaches too, or it would be confusing for the user if the numbers didn't change after they had acknowledged them.

3. The orange danger circle would need to be updated to flash for unacknowledged breaches only.

4. The orange dashboard badge would also need to be updated to only show unacknowledged breaches.

Implementation

A description of how the solution would best be implemented in the code base

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

Additional context

Add any other context or screenshots about the feature request here.

[kat-ms]

Am taking a look at this now (have been fiddling more than actually implementing haha). Few thoughts..

• Like the idea of using the cards to pop up some sort of modal specific to the card. The suggestion on an older version of the issue (Make breaches acknowledgeable #3930) was to have an 'acknowledge all' button on the list page). I think an acknowledge all would be easier for users, but also more prone to careless 'dismissal' 🤔
• Also trying to figure out how to make it 'look' clickable (maybe it can just be a secret feature 😆 - other thought was to make a button appear under the card, but then that's kind of ugly).
• I think cumulative breaches can't really be 'acknowledged' (as it's just a running tally of how long the temperature was above/below a threshold over the date window?) - so will just leave those unclickable.

[adamdewey]

Hey Kat - nice!

• I think it's fine to not be super obvious that it can be clicked for now - as it's the same for cold chain SA. You don't know you can long press the orange thing unless you read the manual I guess?

• I think an acknowledge all approach is fine too, as long as it's per sensor!

• The cumulative thing - perhaps we can just leave that for now. In future it might be nice if the user can reset the cumulative status manually (e.g. if they've thrown out expired stock and replenished and want to start monitoring cumulative total again from now)