Add message if sync settings fails

wilr · Aug 2, 2021 · 564cfdb · 564cfdb
1 parent 609f508
commit 564cfdb
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/Service/AlgoliaService.php b/src/Service/AlgoliaService.php
@@ -144,12 +144,12 @@ public function environmentizeIndex($indexName)
      *
      * This runs automatically on dev/build operations.
      */
-    public function syncSettings()
+    public function syncSettings(): bool
     {
         $config = $this->indexes;
 
         if (!$config) {
-            return;
+            return false;
         }
 
         foreach ($config as $index => $data) {
@@ -177,9 +177,14 @@ function ($replica) {
                         if (Director::isDev()) {
                             throw $e;
                         }
+
+                        return false;
                     }
                 }
             }
         }
+
+
+        return true;
     }
 }
diff --git a/src/Tasks/AlgoliaConfigure.php b/src/Tasks/AlgoliaConfigure.php
@@ -23,7 +23,12 @@ class AlgoliaConfigure extends BuildTask
     public function run($request)
     {
         $service = Injector::inst()->get(AlgoliaService::class);
-        $service->syncSettings();
+
+        if ($service->syncSettings()) {
+            echo 'Success.' . PHP_EOL;
+        } else {
+            echo 'An error occurred while syncing the settings. Please check your error logs.'. PHP_EOL;
+        }
 
         echo 'Done.';
     }