Merge pull request #120 from EscolaLMS/bugfix/multiple-users

Removed required author_id
EscolaLMS · Nov 20, 2024 · 61ad41c · 61ad41c
2 parents 0544ad9 + d655c51
commit 61ad41c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Http/Requests/StoreConsultationRequest.php b/src/Http/Requests/StoreConsultationRequest.php
@@ -24,7 +24,7 @@ public function rules(): array
             'description' => ['required', 'string', 'min:3'],
             'duration' => ['nullable', 'string', 'max:80'],
             'image' => ['nullable', 'file', 'image'],
-            'author_id' => ['required', 'integer', 'exists:users,id'],
+            'author_id' => ['integer', 'exists:users,id'],
             'active_from' => ['date'],
             'active_to' => ['date', 'after_or_equal:active_from'],
             'proposed_dates' => ['array'],

diff --git a/tests/APIs/ConsultationStoreApiTest.php b/tests/APIs/ConsultationStoreApiTest.php
@@ -139,6 +139,6 @@ public function testConsultationStoreWithModelFields(): void
     public function testConsultationStoreRequiredValidation(): void
     {
         $response = $this->actingAs($this->user, 'api')->json('POST', $this->apiUrl);
-        $response->assertJsonValidationErrors(['name', 'status', 'description', 'author_id']);
+        $response->assertJsonValidationErrors(['name', 'status', 'description']);
     }
 }