Skip to content
This repository has been archived by the owner on Jul 16, 2021. It is now read-only.

[Proposal] Add text/plain part to MailMessage #1240

Open
igorsantos07 opened this issue Jun 26, 2018 · 1 comment
Open

[Proposal] Add text/plain part to MailMessage #1240

igorsantos07 opened this issue Jun 26, 2018 · 1 comment

Comments

@igorsantos07
Copy link

The MailMessage class has a very similar API to Mailable, except it's simpler to use (and thus, is sampled on the Notifications guide).
However, it's missing an important part of emails for deliverability: the text/plain part.

Sure, it can be generated using the Markdown parser, but it becomes something not easy to be done when you need to use custom templates.
Mailable solves that by having all three methods: view(), text() and markdown(), but the lack of MailMessage::text() troubles some users (me included).

@taylorotwell
Copy link
Member

I suggest returning a Mailable from your notifications toMail method.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants