Add remote response notification for Loop#504
Conversation
TestTest both with just this branch, and with this branch updated with the latest version of dev (4.5.0) locally.
The LoopFollow plot reflects the remote entries once the data is uploaded from Loop to Nightscout and then downloaded from Nightscout to LoopFollow. The comment for this PR indicates: Next step will be a modification on the Loop side for this to return a message via APNS to the LoopFollow phone. |
marionbarker
left a comment
There was a problem hiding this comment.
Approve from "casual" review and test. Relying on bjorkert's review.
Test SummaryTest with Nightscout PR 19 applied to the Loop app in addition to this version of LoopFollow. ✅ All tests succeeded for both LoopFollow and LoopCaregiver. Test LoopCaregiver and LoopFollowBuild Loop onto my personal phone with the code in Nightscout PR 19 included. LoopCaregiver Test (success)
LoopFollow Test (success)
Check LoopCaregiver - it immediately updates to display latest glucose and treatments from LoopFollow. Graphic showing APNS messages on LoopFollow phoneThe graphic below shows the messages as seen on the LoopFollow phone running 4.5.1 with the Loop phone running the code in Nightscout PR 19. |

Accept APNS responses from looping phone for remote carbs/bolus
Summary
This change makes the app properly accept APNS notifications sent back from a looping phone when a remote carbs and/or bolus request is issued. It ensures the notification path is valid, properly configured, and aligned with the remote-command settings flow so the response can be received reliably.
Details
Adds supporting adjustments in the remote settings and notification handling flow to reduce failure cases in end‑to‑end remote responses.