Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed an issue where all emails where missing the body #4285

Merged

Conversation

valadas
Copy link
Contributor

@valadas valadas commented Nov 7, 2020

While working on the Feedback module the messages where missing their body, then I tried creating new users and those emails where also missing the body. Looks like the CoreMailProvider was missing setting the body to the emails.

This PR solves that.

While working on the Feedback module the messages where missing their body, then I tried creating new users and those emails where also missing the body. Looks like the CoreMailProvider was missing setting the body to the emails.

This PR solves that.
@valadas valadas added this to the 9.8.0 milestone Nov 7, 2020
Copy link
Contributor

@david-poindexter david-poindexter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch - thanks @valadas 🎉

@mitchelsellers mitchelsellers merged commit 971faf3 into dnnsoftware:release/9.8.0 Nov 8, 2020
@valadas valadas deleted the mail-missing-body branch April 14, 2022 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants