-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Description
Is your feature request related to a problem? Please describe.
The pump event history detailed view is complete but can be hard to read.
Describe the solution you'd like
Modify the formatting for the pump event history so the detailed view is easier to read.
Describe alternatives you've considered
N/A
Additional context
The composite below shows the details for an automatic bolus that completed (isMutable: false, automatic: true)
- on the left is the current version
- on the right is one possible pretty-print version
The code for this version exists in the pretty-print-pump-events branch for Loop and LoopKit in the loopandlearn repository. The respective commits are posted at the bottom of this message for review.
More Options
In addition to this example format change for the detailed pump event, the following options could also be considered:
- Add an indication of either Automatic or Manual. It might be best to add "Manual" to a given Bolus or Temp Basal indication and leave automatic deliveries unmodified. This could show up on the main event history screen in addition to the detailed event screen.
- Several variables are nil at this point, not sure if it is worth making the scheduled basal rate or description available for use
- The time is currently local on the main pump event screen and switches to UTC for the detailed screen. A local start time could be added to the display. A duration could also be added in addition to the current UTC start and end time.
Example Code Modification
The commits used for the graphic above are found here:
- LoopKit change: loopandlearn/LoopKit@286381b
- Loop change: loopandlearn@d47f675