Skip to content

Add link to Dexcom app in G7 settings #21

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

Merged
merged 1 commit into from
Apr 6, 2024
Merged

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