docs: add todo comment

src/Abstracts/Jobs/Job.php

@@ -8,6 +8,10 @@
 use Illuminate\Queue\InteractsWithQueue;
 use Illuminate\Queue\SerializesModels;
 
+// TODO: BC
+// remove "implements ShouldQueue"
+// from here and add let the child classes decide if they want to use the queue or not
+// Also maybe move trait to the child classes, so they can decide if they want to use it or not?
 abstract class Job implements ShouldQueue
 {
     use Dispatchable;

src/Abstracts/Providers/AuthServiceProvider.php

@@ -6,6 +6,11 @@
 
 abstract class AuthServiceProvider extends LaravelAuthServiceProvider
 {
+    // TODO: BC
+    // https://laravel.com/docs/10.x/upgrade#register-policies
+    // parent::boot() should be removed from
+    // App\Containers\AppSection\Authentication\Providers\AuthServiceProvider::boot()
+
     /**
      * The policy mappings for the application.
      *

src/Services/Response.php

@@ -55,6 +55,8 @@ private function defaultResourceName(): string
         }
 
         if (!empty($this->data) && 'collection' === $this->determineDataType($this->data)) {
+            // TODO: there was a problem $this->data->first() but I cant remember. It had to do with the data being an array
+            // also check AbstractTransformer where we also do this check and use the first item. we also have the same problem there
             $firstItem = $this->data->first();
             if ($firstItem instanceof HasResourceKey) {
                 return $firstItem->getResourceKey();