[Fix] Let user specify (tenant aware) routes (#9)

* Move mail routes to class and update docs

* Fix downloading attachments

* Fix styling

---------

Co-authored-by: Baspa <[email protected]>

README.md

@@ -61,6 +61,14 @@ Optionally, you can publish the views using
 php artisan vendor:publish --tag="filament-mails-views"
 ```
 
+Add the routes to your `web.php` file:
+
+```php
+use Vormkracht10\FilamentMails\Facades\FilamentMails;
+
+FilamentMails::routes();
+```
+
 Then add the plugin to your `PanelProvider`
 
 ```php
@@ -73,6 +81,22 @@ public function panel(Panel $panel): Panel
 }
 ```
 
+### Tenant middleware / route protection
+
+If you want to protect the mail routes with your (tenant) middleware, you can do so by adding the routes to the `tenantRoutes`:
+
+```php
+use Vormkracht10\FilamentMails\FilamentMailsPlugin;
+use Vormkracht10\FilamentMails\Facades\FilamentMails;
+
+public function panel(Panel $panel): Panel
+{
+    return $panel
+        ->plugin(FilamentMailsPlugin::make())
+        ->tenantRoutes(fn() => FilamentMails::routes());
+}
+```
+
 > [!IMPORTANT]
 > For setting up the webhooks to register mail events, please look into the README of [Laravel Mails](https://github.com/vormkracht10/laravel-mails), the underlying package that powers this package.
 

resources/views/mails/download.blade.php

@@ -1,2 +1,8 @@
-<a type="button" href="{{ route('mail.download', $getState()->id) }}"
+<a type="button"
+    href="{{ route('filament.' . Filament\Facades\Filament::getCurrentPanel()->getId() . '.mails.attachment.download', [
+        'tenant' => Filament\Facades\Filament::getTenant(),
+        'mail' => $getState()->mail_id,
+        'attachment' => $getState()->id,
+        'filename' => $getState()->filename,
+    ]) }}"
     class="rounded-md bg-white px-3.5 py-2.5 text-sm font-semibold cursor-pointer text-gray-900 shadow-sm ring-1 ring-inset ring-gray-300 hover:bg-gray-50">Download</a>

resources/views/mails/preview.blade.php

@@ -1,5 +1,6 @@
 <div class="w-full h-screen">
-    <iframe src="{{ route('mail.preview', ['mail' => $mail->id]) }}" class="w-full h-full max-w-full"
-        style="width: 100vw; height: 100vh; border: none;">
+    <iframe
+        src="{{ route('filament.' . Filament\Facades\Filament::getCurrentPanel()->getId() . '.mails.preview', ['tenant' => Filament\Facades\Filament::getTenant(), 'mail' => $mail->id]) }}"
+        class="w-full h-full max-w-full" style="width: 100vw; height: 100vh; border: none;">
     </iframe>
 </div>

src/Controllers/MailDownloadController.php

@@ -2,20 +2,25 @@
 
 namespace Vormkracht10\FilamentMails\Controllers;
 
-use Illuminate\Http\Request;
 use Illuminate\Routing\Controller;
 use Illuminate\Support\Facades\Storage;
 use Vormkracht10\Mails\Models\MailAttachment;
 
 class MailDownloadController extends Controller
 {
-    public function __invoke(Request $request)
+    public function __invoke(string $tenant, string $mail, string $attachment, string $filename)
     {
         /** @var MailAttachment $attachment */
-        $attachment = MailAttachment::find($request->attachment);
+        $attachment = MailAttachment::find($attachment);
 
-        $file = Storage::disk($attachment->disk)->get($attachment->uuid);
+        $file = Storage::disk($attachment->disk)->path($attachment->uuid);
 
-        return response()->download($file);
+        return response()->download(
+            file: $file,
+            name: $filename,
+            headers: [
+                'Content-Type' => $attachment->mime,
+            ]
+        );
     }
 }

src/FilamentMails.php

@@ -2,4 +2,15 @@
 
 namespace Vormkracht10\FilamentMails;
 
-class FilamentMails {}
+use Illuminate\Support\Facades\Route;
+use Vormkracht10\FilamentMails\Controllers\MailDownloadController;
+use Vormkracht10\FilamentMails\Controllers\MailPreviewController;
+
+class FilamentMails
+{
+    public static function routes()
+    {
+        Route::get('mails/{mail}/preview', MailPreviewController::class)->name('mails.preview');
+        Route::get('mails/{mail}/attachment/{attachment}/{filename}', MailDownloadController::class)->name('mails.attachment.download');
+    }
+}

src/FilamentMailsServiceProvider.php

@@ -56,8 +56,6 @@ public function configurePackage(Package $package): void
         if (file_exists($package->basePath('/../resources/views'))) {
             $package->hasViews(static::$viewNamespace);
         }
-
-        $package->hasRoute('web');
     }
 
     public function packageRegistered(): void {}
@@ -113,8 +111,7 @@ protected function getAssets(): array
      */
     protected function getCommands(): array
     {
-        return [
-        ];
+        return [];
     }
 
     /**