Comments on #167

app/Listeners/ScheduleListener.php

@@ -18,6 +18,9 @@ class ScheduleListener
      */
     public function handle($event)
     {
+        // TODO: this check is also done, when registering the listener EventServiceProvider.php
+        // it can probably be removed here safely which would spare a database call on each request
+
         // only run when internal scheduling is enabled (defensive)
         $method = Setting::where('key', 'app.schedule_method')->value('value') ?? 'internal';
         if ($method !== 'internal') {

app/Providers/EventServiceProvider.php

@@ -29,7 +29,7 @@ public function boot(): void
             }
         }
 
-        // TODO: read where to put these
+        // TODO: read where to put these or ask in the forums
         // it seems to work here, but where is the apropriate place?
         // Kernel::schedule did not work
         Schedule::command('caldav:sync')