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

TRC debug info #335

Merged
merged 1 commit into from
Dec 8, 2024
Merged

TRC debug info #335

merged 1 commit into from
Dec 8, 2024

Conversation

bjorkert
Copy link
Contributor

@bjorkert bjorkert commented Dec 6, 2024

Added information about device token, production env, team id and bundle id in the bottom of the Remote/TRC screen.

I verified that switching build method for Trio changes the production env boolean without restarting LoopFollow. Please note, the production env bool is in the profile and the profile information is refreshed every 10 minutes.

CleanShot 2024-12-06 at 21 12 42@2x

@bjorkert bjorkert requested a review from marionbarker December 6, 2024 20:17
@bjorkert bjorkert changed the title Trc debug info TRC debug info Dec 6, 2024
@marionbarker
Copy link
Collaborator

marionbarker commented Dec 8, 2024

Status

Initial test is good. Hold on merging until I get a version of Trio-dev that uses the production environment.

Testing

First I made sure the LoopFollow dev, commit 0d79ca9 was working with Trio-dev commit 5afb7e4b1, created from Xcode.
Then built this version
The Debug display is present and Production Env is false.

Make sure remote commands go through as expected.

Install the most recent Trio-dev from TestFlight (older version) fbdc3a2
It does not update the Production Env to true and the remote commands do not go through but then it is an older version of TestFlight.

Ask for help on getting a production version ready to test. << figured this out - helps to read the directions.

Copy link
Collaborator

@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.

Approve the changes but want to test with a Production Environment before merging.

@marionbarker
Copy link
Collaborator

I read the fine print in the original comment.

  • I loaded from TestFlight again
  • I changed a basal rate, CR and ISF to make sure the profile updated for NS
  • I waited long enough (>10 minutes) for the LoopFollow to pick up the information from the new profile
  • Production Env now reads true
  • Remote commands went through promptly.

@marionbarker marionbarker merged commit 5693828 into dev Dec 8, 2024
@marionbarker marionbarker deleted the trc-info branch December 13, 2024 21:13
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