Conversation
RoduanKD
left a comment
RoduanKD
left a comment
There was a problem hiding this comment.
I've added some changes here and there. @LanaAlrayes please fix them so I can merge this one.
| */ | ||
| public function index() | ||
| { | ||
| $supports = Support::all(); |
There was a problem hiding this comment.
please use pagination and sort them by id descendingly
| 'title' => 'required|string|min:3|max:255', | ||
| 'details' => 'required|text|min:3|max:255', | ||
| ]); | ||
| $request = new Support(); |
There was a problem hiding this comment.
Please use Support::create instead
There was a problem hiding this comment.
and don't forget about the fillable property
app/Models/User.php
Outdated
|
|
||
| public function support() | ||
| { | ||
| return $this->hasMany(Support::class); |
There was a problem hiding this comment.
You need here to specify the column name based on the user role. and it's best to call this one tickets instead of support. (ignore this until @Aubada10 is done with roles & permissions)
| $table->text('details'); | ||
| $table->integer('status'); | ||
| $table->foreignId('owner_id')->constrained('users'); | ||
| $table->foreignId('employee_id')->constrained('users'); |
| @endforeach | ||
| </tbody> | ||
| </table> | ||
| </div> |
There was a problem hiding this comment.
don't forget to add pagination links here
| <p class="text-muted">Details: {{ $support->details }}</p> | ||
| </div> | ||
| </div> | ||
| </div> |
There was a problem hiding this comment.
we'll need also to have comments (ignore this until the comments pull request is ready)
2 participants
No description provided.