Reboot on config save and remove local hotword settings

Romkabouter · Apr 21, 2021 · dcf5104 · dcf5104
1 parent 4c2b827
commit dcf5104
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 11 deletions.
diff --git a/PlatformIO/src/General.hpp b/PlatformIO/src/General.hpp
@@ -364,7 +364,8 @@ void loadConfiguration(const char *filename, Config &config) {
     config.brightness = doc.getMember("brightness").as<int>();
     device->updateBrightness(config.brightness);
     config.hotword_brightness = doc.getMember("hotword_brightness").as<int>();
-    config.hotword_detection = doc.getMember("hotword_detection").as<int>();
+    //config.hotword_detection = doc.getMember("hotword_detection").as<int>();
+    config.hotword_detection = 1;
     config.volume = doc.getMember("volume").as<int>();
     device->setVolume(config.volume);
     config.gain = doc.getMember("gain").as<int>();
@@ -405,9 +406,9 @@ void saveConfiguration(const char *filename, Config &config) {
     doc["volume"] = config.volume;
     doc["gain"] = config.gain;
     if (serializeJson(doc, file) == 0) {
-         Serial.println(F("Failed to write to file"));
+        Serial.println(F("Failed to write to file"));
     }
     file.close();
-    loadConfiguration(filename, config);
-    audioServer.disconnect();
+    Serial.println("Configuration saved! Rebooting");
+    ESP.restart();    
 }
diff --git a/PlatformIO/src/Satellite.cpp b/PlatformIO/src/Satellite.cpp
@@ -87,6 +87,9 @@
     - Now scans for strongest AP 
     - Added INMP441 & INMP441MAX98357A
     - Siteid can now be configured in webui
+   v7.5.1
+    - Saving the configuration now reboots due to instablility issue in disconnecting MQTT
+    - Remove hotword method from UI and hard setting to REMOTE until local works again
 
 * ************************************************************************ */
 

diff --git a/PlatformIO/src/index_html.h b/PlatformIO/src/index_html.h
@@ -101,13 +101,6 @@ input::-moz-focus-inner,input::-moz-focus-outer {border: 0;}
         <span class="range-slider__value">0</span>
       </div>
     </div>
-    <div class="input-container">
-      <label for="hotword_detection">Hotword detection:&nbsp;</label>
-      <select name="hotword_detection">
-        <option value="0" %HW_LOCAL%>Local</option>
-        <option value="1" %HW_REMOTE%>Remote</option>
-      </select>
-    </div>
     <div class="input-container">
       <label for="gain">Gain:&nbsp;</label>
       <div class="range-slider">