Skip to content

Conversation

@JamieKeene
Copy link
Contributor

I found myself missing the 'Open app' button from the G6, and that my muscle memory kept expecting it to be there, so in this PR I've added it into the G7 management screen.

I opted to create a new section rather than include it with the other management buttons beneath to try to prevent users accidentally tapping the more destructive actions.

I found myself missing the 'Open app' button from the G6, and that my muscle memory kept expecting it to be there, so in this PR I've added it into the G7 management screen.

I opted to create a new section rather than include it with the other management buttons beneath to try to prevent users accidentally tapping the more destructive actions.
@marionbarker
Copy link
Contributor

I tested this PR. It works as expected.

The graphics below show the G7 screen before (left) and after (right) this modification. The new section is highlighted by the red rectangle.
g7-pr-21

@JamieKeene
Copy link
Contributor Author

@ps2 Please let me know if I need to do anything else to get this merged.

@marionbarker
Copy link
Contributor

Thinking about how I use a tap on Glucose on the main Loop screen:

  1. My primary use is a quick-glance at how long I have until the sensor expires (getting this from G7 app, requires multiple taps)
  2. With this PR, I could start with a tap on the glucose in main Loop screen followed by tap on Open Dexcom App to acknowledge alerts
    • Right now, I keep the two app icons side-by-side on my phone for easier access to G7 App to acknowledge alerts

If this modification were moved to line 77 (from it's current position of line 126), then tapping the main Loop screen CGM display provides both

  • easy access to dates for current sensor
  • easy access to G7 app needed to acknowledge alerts

The graphic below shows this modification:

g7sensorkit-pr21-modification

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants