Skip to content

osaid-keymanager v3#63

Open
jwoglom wants to merge 27 commits into
mainfrom
jwoglom/osaid-keymanager
Open

osaid-keymanager v3#63
jwoglom wants to merge 27 commits into
mainfrom
jwoglom/osaid-keymanager

Conversation

@jwoglom
Copy link
Copy Markdown
Collaborator

@jwoglom jwoglom commented May 23, 2026

Add osaid-keymanager with osaid-keymanager-v2 fixes, as a branch on loopandlearn/OmnipodKit (to avoid needing to fetch my fork as an additional git origin)

@marionbarker
Copy link
Copy Markdown
Contributor

Status

NOTE to people not part of the private beta test who are reading these PR with interest.

  • You cannot use this code to pair an O5 pod
  • This version only gives you the option to control Classic (Eros) or DASH
  • There are additional instructions only available to members of the private beta test
  • Please let us test and analyze results
  • We are working as fast as we can

Previous PR

Refer to the original version of this PR here:

In the interim, some issues were reported, updated and fixed in the the jwoglom repo - which was later deleted and reforked so I can't link to the PR where the updates were reported, tested, approved and merged into the jwoglom/osaid-keymanager branch.

This comparison replicates the changes that were made recently.

The new jwoglom repo is a new fork

  • The required branch (for this PR) was pushed directly to the loopandlearn upstream repository for OmnipodKit

Previous testing

There were detailed notes here (but this was a different version of the code)

Updated testing

From memory -

  • The bug where you request a cert and get one then cannot proceed to pairing with an O5 is fixed.
  • Some localizations were updated.
  • The Pod Certificate rows is now found just above Pod Diagnostics but only when O5 is the selected Pod type
  • Some action buttons were removed that were not needed
  • The Pump Manager details are updated as soon as a cert is obtained, even when no Pod is paired

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