Skip to content

Conversation

nunomaduro
Copy link
Member

This pull request fixes markdown access to $message variable on template, by simply lazy render the markdown on the moment the $message variable as already available as view data.

@dakira
Copy link
Contributor

dakira commented May 19, 2023

@nunomaduro This solution is beautiful! I really just learned a lot by solving this myself and now look at your solution. Thanks and great work!

@nunomaduro nunomaduro changed the title [10.x] Fixes markdown access to $message variable on template [10.x] Adds inline attachments support for markdown mailables May 19, 2023
@nunomaduro
Copy link
Member Author

nunomaduro commented May 19, 2023

@dakira Could you please conduct some local tests to verify that this feature works as expected in your use case and does not break any existing behaviors? Thank you for helping!

@nunomaduro nunomaduro self-assigned this May 19, 2023
@nunomaduro nunomaduro requested a review from taylorotwell May 21, 2023 00:04
@nunomaduro nunomaduro marked this pull request as ready for review May 22, 2023 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants