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

Home View shows "No Pod" or "Signal Loss" despite insulin delivery system removal #283

Closed
dnzxy opened this issue Jun 8, 2024 · 0 comments · Fixed by #285
Closed

Home View shows "No Pod" or "Signal Loss" despite insulin delivery system removal #283

dnzxy opened this issue Jun 8, 2024 · 0 comments · Fixed by #285
Labels
bug Something isn't working

Comments

@dnzxy
Copy link
Contributor

dnzxy commented Jun 8, 2024

Describe the bug

This issue became apparent while testing #263 and #277 . When removing a pump, the Home View shows "No Pod" or "Signal Loss" despite removing the insulin delivery system via the pump manager UI.

To Reproduce

Steps to reproduce the behavior:

  1. Remove pod/pump via pump manager UI
  2. Go back to home view
  3. Pump status section shows "No Pod" or "Signal Loss"
  4. Tapping on that notification takes user to pump manager UI to select a new insulin delivery device

Expected behavior

When removing the pump, Home View must display "Add Pump", because it was removed.

Technical Details

This is connected to not properly observing updates of DeviceManager/PumpStatusHighlight when removing the delivery system.

@dnzxy dnzxy added bug Something isn't working needs-triage and removed needs-triage labels Jun 8, 2024
@dnzxy dnzxy assigned dnzxy and unassigned dnzxy Jun 8, 2024
@dnzxy dnzxy added this to Trio Jun 8, 2024
@dnzxy dnzxy moved this to In progress in Trio Jun 8, 2024
@bjornoleh bjornoleh linked a pull request Jun 8, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this from In progress to Done in Trio Jun 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant