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

[Mailer] Add Message-Id to SentMessage when sending an email #33967

Merged
merged 2 commits into from Oct 13, 2019

Conversation

@fabpot
Copy link
Member

fabpot commented Oct 12, 2019

Q A
Branch? 4.4
Bug fix? yes-ish
New feature? yes-ish as well
Deprecations? no
Tickets fixes #33681
License MIT
Doc PR -

This adds SentMessage::getMessageId() to retrieve the message id as generated internally OR by the provider sending the email.

@fabpot
Copy link
Member Author

fabpot commented Oct 12, 2019

Failed tests are ok (master branch only)

@fabpot fabpot force-pushed the fabpot:mailer-message-id branch from 157f7e9 to d97d1f9 Oct 13, 2019
@fabpot fabpot merged commit d97d1f9 into symfony:4.4 Oct 13, 2019
1 of 3 checks passed
1 of 3 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
fabbot.io Your code looks good.
Details
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.4 Oct 27, 2019
This was referenced Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

5 participants
You can’t perform that action at this time.