Update meal label, proposed remedy for Issue #2168 by marionbarker · Pull Request #2300 · LoopKit/Loop
Status
I suggest this button change be accepted for now with potential updates coming at a later time.
- Final wording on the button is "Save Carbs & Deliver"
Discussion
I had a discussion with some Tidepool folks:
- They were concerned that the button change alone would not be sufficient to solve Avoid unnoticed carb entries #2168
- I agreed to explore adding messages that indicated that "If carbs were added, they were saved"
- After looking into this, my suggestion is to just make the button change for now
There are 2 places where added messages can happen
- Uncertain comms modal alert
- Bolus Issue alert
One negative of this approach is the carbs saved message would be displayed for any bolus or uncertain comms issues.
Option: Modify uncertain comms modal message
If I implemented this, it could look like the graphic below, where the new text is underlined to make it easy to find.
Option: Add to the Bolus Issue alerts
Default screens are seen in the previous comment for some Bolus Issue alerts.
- I tried adding the sentence to the alert in the body of the message but the message truncates and important information is lost
- I then tried modifying the title of the alert, i.e., alert title changed to "Bolus Issue (Carbs Saved)" and was not convinced and even this is truncated with larger text sizes
