[CGM Config View] - Nightscout button

Add a link to configure Nightscout if not defined. Improve minor code
mountrcg · May 19, 2024 · d7c7b6d · d7c7b6d
1 parent 5011290
commit d7c7b6d
Show file tree

Hide file tree

Showing 4 changed files with 40 additions and 21 deletions.
diff --git a/FreeAPS/Sources/Modules/CGM/CGMStateModel.swift b/FreeAPS/Sources/Modules/CGM/CGMStateModel.swift
@@ -138,18 +138,20 @@ extension CGM {
                 .store(in: &lifetime)
         }
 
-        func displayNameOfApp() -> String {
+        func displayNameOfApp() -> String? {
+            guard cgmManager != nil else { return nil }
             var nameOfApp = "Open Application"
             switch cgmManager.cgmGlucoseSourceType {
             case .plugin:
                 nameOfApp = "Open " + (cgmManager.cgmManager?.localizedTitle ?? "Application")
             default:
-                nameOfApp = "Open " + (cgmManager.cgmGlucoseSourceType.displayName ?? "Application")
+                nameOfApp = "Open " + cgmManager.cgmGlucoseSourceType.displayName
             }
             return nameOfApp
         }
 
         func urlOfApp() -> URL? {
+            guard cgmManager != nil else { return nil }
             switch cgmManager.cgmGlucoseSourceType {
             case .plugin:
                 return cgmManager.cgmManager?.appURL

diff --git a/FreeAPS/Sources/Modules/CGM/View/CGMRootView.swift b/FreeAPS/Sources/Modules/CGM/View/CGMRootView.swift
@@ -46,29 +46,29 @@ extension CGM {
                         }
                     }
 
-                    if let cgmFetchManager = state.cgmManager {
-                        if let appURL = state.urlOfApp()
-                        {
-                            Section {
-                                Button {
-                                    UIApplication.shared.open(appURL, options: [:]) { success in
-                                        if !success {
-                                            self.router.alertMessage
-                                                .send(MessageContent(content: "Unable to open the app", type: .warning))
-                                        }
+                    if let appURL = state.urlOfApp()
+                    {
+                        Section {
+                            Button {
+                                UIApplication.shared.open(appURL, options: [:]) { success in
+                                    if !success {
+                                        self.router.alertMessage
+                                            .send(MessageContent(content: "Unable to open the app", type: .warning))
                                     }
                                 }
-
-                                label: {
-                                    Label(state.displayNameOfApp(), systemImage: "waveform.path.ecg.rectangle").font(.title3) }
-                                    .frame(maxWidth: .infinity, alignment: .center)
-                                    .buttonStyle(.bordered)
                             }
-                            .listRowBackground(Color.clear)
-                        } else if state.cgmCurrent.type == .nightscout && state.url != nil {
+
+                            label: {
+                                Label(state.displayNameOfApp() ?? "-", systemImage: "waveform.path.ecg.rectangle").font(.title3) }
+                                .frame(maxWidth: .infinity, alignment: .center)
+                                .buttonStyle(.bordered)
+                        }
+                        .listRowBackground(Color.clear)
+                    } else if state.cgmCurrent.type == .nightscout {
+                        if let url = state.url {
                             Section {
                                 Button {
-                                    UIApplication.shared.open(state.url!, options: [:]) { success in
+                                    UIApplication.shared.open(url, options: [:]) { success in
                                         if !success {
                                             self.router.alertMessage
                                                 .send(MessageContent(content: "No URL available", type: .warning))
@@ -80,6 +80,18 @@ extension CGM {
                                     .buttonStyle(.bordered)
                             }
                             .listRowBackground(Color.clear)
+                        } else {
+                            Section {
+                                Button {
+                                    state.showModal(for: .nighscoutConfigDirect)
+                                    // router.mainSecondaryModalView.send(router.view(for: .nighscoutConfigDirect))
+                                }
+                                label: { Label("Config Nightscout", systemImage: "waveform.path.ecg.rectangle").font(.title3)
+                                }
+                                .frame(maxWidth: .infinity, alignment: .center)
+                                .buttonStyle(.bordered)
+                            }
+                            .listRowBackground(Color.clear)
                         }
                     }
 

diff --git a/FreeAPS/Sources/Modules/NightscoutConfig/View/NightscoutConfigRootView.swift b/FreeAPS/Sources/Modules/NightscoutConfig/View/NightscoutConfigRootView.swift
@@ -5,6 +5,7 @@ import Swinject
 extension NightscoutConfig {
     struct RootView: BaseView {
         let resolver: Resolver
+        let displayClose: Bool
         @StateObject var state = StateModel()
         @State var importAlert: Alert?
         @State var isImportAlertPresented = false
@@ -153,6 +154,7 @@ extension NightscoutConfig {
             .onAppear(perform: configureView)
             .navigationBarTitle("Nightscout Config")
             .navigationBarTitleDisplayMode(.automatic)
+            .navigationBarItems(leading: displayClose ? Button("Close", action: state.hideModal) : nil)
             .alert(isPresented: $isImportAlertPresented) {
                 importAlert!
             }

diff --git a/FreeAPS/Sources/Router/Screen.swift b/FreeAPS/Sources/Router/Screen.swift
@@ -7,6 +7,7 @@ enum Screen: Identifiable, Hashable {
     case settings
     case configEditor(file: String)
     case nighscoutConfig
+    case nighscoutConfigDirect
     case pumpConfig
     case pumpSettingsEditor
     case basalProfileEditor
@@ -55,7 +56,9 @@ extension Screen {
         case let .configEditor(file):
             ConfigEditor.RootView(resolver: resolver, file: file)
         case .nighscoutConfig:
-            NightscoutConfig.RootView(resolver: resolver)
+            NightscoutConfig.RootView(resolver: resolver, displayClose: false)
+        case .nighscoutConfigDirect:
+            NightscoutConfig.RootView(resolver: resolver, displayClose: true)
         case .pumpConfig:
             PumpConfig.RootView(resolver: resolver)
         case .pumpSettingsEditor: