Skip to content

[Mailer] Mailjet Add ability to pass custom headers to API #37934

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

Merged
merged 1 commit into from
Aug 27, 2020

Conversation

tcheymol
Copy link
Contributor

Q A
Branch? master
Bug fix? no
New feature? yes
Deprecations? no
Tickets no
License MIT
Doc PR no

Mailjet mailer now forwards headers from original email removes forbidden headers, and sends them to Maijlet Api

Additionally (see https://symfony.com/releases):

  • Always add tests and ensure they pass.
  • Never break backward compatibility (see https://symfony.com/bc).
  • Bug fixes must be submitted against the lowest maintained branch where they apply
    (lowest branches are regularly merged to upper ones so they get the fixes too.)
  • Features and deprecations must be submitted against branch master.
    -->

@fabpot
Copy link
Member

fabpot commented Aug 26, 2020

To fix the tests, you should probably bump the min version for symfony/mime to ^5.2.
`

@tcheymol
Copy link
Contributor Author

@fabpot Do you mean the version in the Mailer component's composer.json or inside Mailjet Mailer's composer.json, in the Mailer component, the version is already ^5.2

@fabpot
Copy link
Member

fabpot commented Aug 26, 2020

I mean Mailjet Mailer's composer.json

@fabpot
Copy link
Member

fabpot commented Aug 27, 2020

Thank you @tcheymol.

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