Adds a line in the notification email to manage its subscriptions.

Untill now a subscriber wasn't able to manage its subscription because there was no link anywhere, so it was forced to unsubscribe and then subscribe again. Now the link is in the notification email, after 'Unsubscribe at'. Related to cachethq#2887
uxen-ab · Jan 30, 2018 · 5ae13fb · 5ae13fb
1 parent 1fec3e0
commit 5ae13fb
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/app/Notifications/Component/ComponentStatusChangedNotification.php b/app/Notifications/Component/ComponentStatusChangedNotification.php
@@ -88,7 +88,9 @@ public function toMail($notifiable)
                     ->greeting(trans('notifications.component.status_update.mail.subject'))
                     ->line($content)
                     ->action(trans('notifications.component.status_update.mail.action'), cachet_route('status-page'))
-                    ->line(trans('cachet.subscriber.unsubscribe', ['link' => cachet_route('subscribe.unsubscribe', $notifiable->verify_code)]));
+                    ->line(trans('cachet.subscriber.unsubscribe', ['link' => cachet_route('subscribe.unsubscribe', $notifiable->verify_code)]))
+                    //TODO: Translate the text below
+                    ->line('Manage your subscriptions at ' . cachet_route('subscribe.manage', $notifiable->verify_code));
     }
 
     /**

diff --git a/app/Notifications/Incident/NewIncidentNotification.php b/app/Notifications/Incident/NewIncidentNotification.php
@@ -79,7 +79,9 @@ public function toMail($notifiable)
                     ->line($content)
                     ->action(trans('notifications.incident.new.mail.action'), cachet_route('incident', [$this->incident]))
                     ->line($this->incident->message)
-                    ->line(trans('cachet.subscriber.unsubscribe', ['link' => cachet_route('subscribe.unsubscribe', $notifiable->verify_code)]));
+                    ->line(trans('cachet.subscriber.unsubscribe', ['link' => cachet_route('subscribe.unsubscribe', $notifiable->verify_code)]))
+                    //TODO: Translate the text below.
+                    ->line('Manage your subscriptions at '. cachet_route('subscribe.manage',  $notifiable->verify_code));
     }
 
     /**

diff --git a/app/Notifications/IncidentUpdate/IncidentUpdatedNotification.php b/app/Notifications/IncidentUpdate/IncidentUpdatedNotification.php
@@ -83,7 +83,10 @@ public function toMail($notifiable)
                     ->line($content)
                     ->action(trans('notifications.incident.update.mail.action'), cachet_route('incident', [$this->update->incident]))
                     ->line($this->update->message)
-                    ->line(trans('cachet.subscriber.unsubscribe', ['link' => cachet_route('subscribe.unsubscribe', $notifiable->verify_code)]));
+                    ->line(trans('cachet.subscriber.unsubscribe', ['link' => cachet_route('subscribe.unsubscribe', $notifiable->verify_code)]))
+                    //TODO: Translate the text below
+                    ->line('Manage your subscriptions at ' . cachet_route('subscribe.manage', $notifiable->verify_code));
+
     }
 
     /**