requested changes from github

app/Http/Controllers/Api/NotesController.php

@@ -8,16 +8,24 @@ use App\Http\Controllers\Controller;
 use App\Models\Asset;
 use Illuminate\Http\Request;
 use Illuminate\Support\Facades\Auth;
+use Illuminate\Validation\Rule;
 
 class NotesController extends Controller
 {
     public function store(Request $request)
     {
+        $validated = $request->validate([
+            'note' => 'required|string|max:500',
+            'type' => [
+                'required',
+                Rule::in(['asset']),
+            ],
+        ]);
+
         $item = Asset::findOrFail($request->input("id"));
         $this->authorize('update', $item);
-
+        // This can be made dynamic by using $request->input('type') to determine which model type to add the note to.
-        $validated = $request->validate([
+        // For now, we are only placing this on Assets
-            'note' => 'required|string|max:500']);
 
         event(new NoteAdded($item, Auth::user(), $validated['note']));
 

app/Models/Note.php

@@ -1,17 +0,0 @@
-<?php
-
-namespace App\Models;
-
-use Illuminate\Database\Eloquent\Model;
-use Illuminate\Database\Eloquent\Relations\MorphTo;
-
-class Note extends Model
-{
-    /**
-     * Get the parent item of a comment
-     */
-    public function commentable(): MorphTo
-    {
-        return $this->morphTo();
-    }
-}