Skip to content
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

direct user where to flip Upload Glucose toggle #338

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

MikePlante1
Copy link
Contributor

@MikePlante1 MikePlante1 commented Jun 29, 2024

Since the Upload Glucose toggle is disabled when using a plugin for the cgm source (Dexcom or LibreTransmitter), display text to direct the user to where they can flip the toggle. If they aren't using a plugin for their cgm source, then this text won't be displayed.

To flip the Upload Glucose toggle, go to ⚙️ > CGM > CGM Configuration

Screenshot 2024-06-29 at 1 36 51 PM

Addresses issue #199

Only display if the selected CGM disables the toggle in Nightscout > Upload.
@marionbarker
Copy link
Contributor

Should this be a request to dev or to alpha

I confirmed that this works as indicated, but it is still really confusing.
The extra message showed up in the NS Upload screen when either G6 or G7 was selected as the CGM.
When I had xDrip4iOS selected as the CGM, the NS glucose toggle worked directly in the NS Upload screen.

It's still really confusing, but I think this is a good-enough work-around for RC1.

@marionbarker marionbarker self-requested a review June 30, 2024 21:29
marionbarker
marionbarker previously approved these changes Jun 30, 2024
Copy link
Contributor

@marionbarker marionbarker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as a bandaid fix.

@Sjoerd-Bo3 Sjoerd-Bo3 changed the base branch from dev to alpha July 2, 2024 13:25
@Sjoerd-Bo3 Sjoerd-Bo3 dismissed marionbarker’s stale review July 2, 2024 13:25

The base branch was changed.

Sjoerd-Bo3
Sjoerd-Bo3 previously approved these changes Jul 2, 2024
Copy link
Contributor

@Sjoerd-Bo3 Sjoerd-Bo3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Sjoerd-Bo3 Sjoerd-Bo3 changed the base branch from alpha to dev July 2, 2024 13:40
@Sjoerd-Bo3 Sjoerd-Bo3 dismissed their stale review July 2, 2024 13:40

The base branch was changed.

@Sjoerd-Bo3 Sjoerd-Bo3 self-requested a review July 2, 2024 13:40
@Sjoerd-Bo3 Sjoerd-Bo3 merged commit 674bacc into nightscout:dev Jul 2, 2024
@MikePlante1 MikePlante1 deleted the upload_glucose branch July 2, 2024 13:54
@MikePlante1 MikePlante1 mentioned this pull request Jul 16, 2024
mountrcg pushed a commit to mountrcg/Trio that referenced this pull request Jul 20, 2024
direct user where to flip Upload Glucose toggle
mountrcg pushed a commit to mountrcg/Trio that referenced this pull request Aug 1, 2024
This was referenced Oct 18, 2024
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.

Add Referral Text for Upload Glucose Icon in Nightscout Config
3 participants