PriceyBot/PriceyBotPanel
Archived
5
0
Fork 0
Code Issues 6 Pull Requests Actions Packages Projects 1 Releases 4 Wiki Activity

feat/admin-send-quote #15

Merged
llama merged 5 commits from feat/admin-send-quote into master 2023-10-09 05:09:57 -04:00
Conversation 5 Commits 5 Files Changed 4 +2 -2
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 9fc26f5125 - Show all commits

2
resources/views/components/sidebar.blade.php
View File

@ -35,7 +35,7 @@
<ul role="list" class="-mx-2 mt-2 space-y-1">
<li>
{{-- TODO: fill with SVG icons to make it pretty --}}
<x-sidebar-nav :href="route('send')" route="send">Send Quote</x-sidebar-nav>
<x-sidebar-nav :href="route('admin.send')" route="send">Send Quote</x-sidebar-nav>
benjamyn marked this conversation as resolved Outdated
llama commented 2023-10-09 05:05:36 -04:00
Outdated
Review
Copy Link

I think this also needs to be /admin/send

I think this also needs to be /admin/send
</li>
<li>
{{-- TODO: fill with SVG icons to make it pretty --}}

2
routes/web.php
View File

@ -27,7 +27,7 @@ Route::view('profile', 'profile')
Route::view('send', 'admin.send')
llama commented 2023-10-09 04:58:30 -04:00
Outdated
Review
Copy Link

can we prefix send with admin/

once we start getting more admin tools, it'll get mess but the rest looks good to me :D

can we prefix send with admin/ once we start getting more admin tools, it'll get mess but the rest looks good to me :D
llama commented 2023-10-09 05:06:19 -04:00
Review
Copy Link

Route::view('admin/send', 'admin.send')

will be better I think

Route::view('admin/send', 'admin.send') will be better I think
->middleware(['auth'])
->name('send');
->name('admin.send');
Route::middleware('guest')->group(function () {
Volt::route('login', 'pages.auth.login')