From 875e627d9d9e049c0dbc6a9619597f42eee90925 Mon Sep 17 00:00:00 2001
From: Mitch Bradley <wmb@firmworks.com>
Date: Wed, 9 Oct 2024 11:33:46 -1000
Subject: [PATCH] Dollar commands are asynchronous by default; helps WebUI

---
 FluidNC/src/ProcessSettings.cpp | 2 --
 FluidNC/src/Settings.h          | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/FluidNC/src/ProcessSettings.cpp b/FluidNC/src/ProcessSettings.cpp
index 350199f6f..f080ac512 100644
--- a/FluidNC/src/ProcessSettings.cpp
+++ b/FluidNC/src/ProcessSettings.cpp
@@ -899,8 +899,6 @@ Error do_command_or_setting(const char* key, const char* value, AuthenticationLe
         }
     }
 
-    protocol_buffer_synchronize();
-
     // First search the yaml settings by name. If found, set a new
     // value if one is given, otherwise display the current value
     try {
diff --git a/FluidNC/src/Settings.h b/FluidNC/src/Settings.h
index 3ce7299aa..25d885191 100644
--- a/FluidNC/src/Settings.h
+++ b/FluidNC/src/Settings.h
@@ -100,7 +100,7 @@ class Command : public Word {
             const char*   grblName,
             const char*   fullName,
             bool (*cmdChecker)(),
-            bool synchronous = true);
+            bool synchronous = false);
 
     // The default implementation of addWebui() does nothing.
     // Derived classes may override it to do something.