Skip to content

Clarify wording for Learn More screen in Uncertain Delivery State#23

Merged
ps2 merged 3 commits into
LoopKit:mainfrom
marionbarker:unable-to-reach
Jan 26, 2024
Merged

Clarify wording for Learn More screen in Uncertain Delivery State#23
ps2 merged 3 commits into
LoopKit:mainfrom
marionbarker:unable-to-reach

Conversation

@marionbarker
Copy link
Copy Markdown
Collaborator

@marionbarker marionbarker commented Jan 11, 2024

Modify wording to be consistent with OmniBLE PR 104 with addition of words about cycling the RileyLink to clarify actions for a user who is in an uncertain delivery state.

@marionbarker marionbarker changed the title Clarify working for Learn More screen in Uncertain Delivery State Clarify wording for Learn More screen in Uncertain Delivery State Jan 11, 2024
@marionbarker
Copy link
Copy Markdown
Collaborator Author

marionbarker commented Jan 12, 2024

I was able to test this for an Eros pod. See the method in the next comment. The wording was modified based on several tests.

This comment was edited on 24-Jan-2024 to remove the old graphics.

@marionbarker
Copy link
Copy Markdown
Collaborator Author

Improved test method:

  1. set Loop to closed-loop with Temp Basal Only dosing strategy and feed it with Nightscout as CGM using an unannounced meal trace (increasing glucose to force frequent TB changes)
  2. phone should be connected to an Eros pod using a rileylink (I'm using one with a Medtronic Antenna to deliberately downgrade RF signal)
  3. When system is operating nominally (green loop), then insert Rileylink in faraday bag and wait for Uncertain Comms screen to appear

@marionbarker
Copy link
Copy Markdown
Collaborator Author

This test used PR #25 (which fixes Loop Issue 2118) in addition to PR 23.

Top row - comes from LoopKit and is not modified.
Middle row - shows top, middle and bottom of the screen addressed by this PR
Bottom row - shows the screen shown after comms are restored (not modified by this PR)

omnikit-pr-23

@ps2 ps2 merged commit 0fdcbf9 into LoopKit:main Jan 26, 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.

2 participants