Refactor all emails to use Markdown views

[jbrooksuk]

Replaces #3015

---

• Every email should be Markdown based as this makes it very easy to implement any dynamically generated Markdown content.
• Every email should contain Unsubscribe and Manage subscription links (this should be a partial).

This should be looked at as soon as #3100 is merged. It's even more important now that GDPR is live.

[nstapelbroek]

After merging #3100 , these notifications need markdown support:

• Notifications/Component/ComponentStatusChangedNotification.php
• Notifications/IncidentUpdate/IncidentUpdatedNotification.php
• Notifications/Schedule/NewScheduleNotification.php
• Notifications/Subscriber/VerifySubscriptionNotification.php
• Notifications/System/SystemTestNotification.php
• Notifications/User/InviteUserNotification.php

[JamesWilson19947]

@jbrooksuk @nstapelbroek any ideas of when this will be implemented or if I can even do this?

It's basically the primary reason we cannot use it as a company, because without the consistent unsubscribe button on the emails, it's not GDPR compliant! :)

[JamesWilson19947]

I'm seriously cool with helping out with this work if you need! 👍
I just don't know exactly how you want the markdown example I made this for the update notifications:

In IncidentUpdatedNotification.php

return (new MailMessage())
                    ->subject(trans('notifications.incident.update.mail.subject'))
                    ->markdown('notifications.incident.update', [
                        'incident'               => $this->update->incident,
                        'content'                => $content,
                        'incidentName'           => $this->update->incident->name,
                        'newStatus'              => $this->update->human_status,
                        'unsubscribeText'        => trans('cachet.subscriber.unsubscribe'),
                        'unsubscribeUrl'         => cachet_route('subscribe.unsubscribe', $notifiable->verify_code),
                        'manageSubscriptionText' => trans('cachet.subscriber.manage_subscription'),
                        'manageSubscriptionUrl'  => cachet_route('subscribe.manage', $notifiable->verify_code),
                    ]);

update.blade.php

@component('mail::message')
# {{ trans('notifications.incident.update.mail.title', ['name' => $incidentName, 'new_status' => $newStatus])  }}

{{ $content }}

Thanks,<br>
{{ Config::get('setting.app_name') }}

@include('notifications.partials.subscription')

@endcomponent

Update:
I did the markdown for

• Notifications/Component/ComponentStatusChangedNotification.php
• Notifications/IncidentUpdate/IncidentUpdatedNotification.php
• Notifications/Schedule/NewScheduleNotification.php

I haven't done the others yet, as they are less important! I'm happy to raise a pull request if you wish?

[jbrooksuk]

Fixed in #3270