Skip to content

put Silence Pod and Pod Diagnostics behind build-time flag#5

Merged
marionbarker merged 1 commit into
silent_pod_and_diagnosticsfrom
diy_flag_for_silence_diagnostics
Mar 23, 2024
Merged

put Silence Pod and Pod Diagnostics behind build-time flag#5
marionbarker merged 1 commit into
silent_pod_and_diagnosticsfrom
diy_flag_for_silence_diagnostics

Conversation

@marionbarker
Copy link
Copy Markdown

  • supports Loop by using DEBUG_FEATURES_ENABLED flag
  • make suspend message more generic

* supports Loop by using DEBUG_FEATURES_ENABLED flag
* make suspend message more generic
@marionbarker
Copy link
Copy Markdown
Author

marionbarker commented Mar 23, 2024

To test while still in the PR / branch diy_flag_for_silence_diagnostics now that this is merged (the SHA below were updated).

  • For Mac build - paste in terminal in LoopWorkspace folder
  • For Browser build - paste at the beginning of customization section of build_loop.yml
    • If testing more than one module, the update-index commands can be added one after the other
    • The git submodule update must come last
git update-index --cacheinfo 160000 5ff65bee566378de23d4d4b32a867c8f8f0c6de6 "OmniBLE"
git submodule update

Then build.

For the iphone version of iAPS that uses submodules - the Silence Pod and Pod diagnostics should be present in the OmniBLE Pod Settings screen because the logic flag is hard-coded to true (right side of graphic below).

For Loop, when DEBUG_FEATURES_ENABLED is commented out of the LoopConfigOverride.xcconfig file, the extra features are not present (left side of graphic below), when that flag is present (default for DIY Loop), the extra features are present (right side of graphic below).

lnl-omnible-pr-5

@marionbarker marionbarker merged commit 5ff65be into silent_pod_and_diagnostics Mar 23, 2024
@marionbarker marionbarker deleted the diy_flag_for_silence_diagnostics branch June 13, 2024 22:41
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.

1 participant