Skip to content

Implement Message Stream for Postmark Mailer #15793

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.

Sign up for GitHub

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

Jump to bottom
Closed
fabpot opened this issue Sep 10, 2021 · 4 comments
Closed

Implement Message Stream for Postmark Mailer #15793

fabpot opened this issue Sep 10, 2021 · 4 comments
Labels
Keep open Mailer
Milestone
5.4

Comments

@fabpot
Copy link
Member

fabpot commented Sep 10, 2021

Q A
Feature PR symfony/symfony#42941
PR author(s) @driesvints
Merged in 5.4
@fabpot fabpot added this to the 5.4 milestone Sep 10, 2021
@xabbuh xabbuh added the Mailer label Oct 4, 2021
@carsonbot
Copy link
Collaborator

Thank you for this issue.
There has not been a lot of activity here for a while. Has this been resolved?

@carsonbot carsonbot removed the Stalled label Nov 12, 2022
@alamirault
Copy link
Contributor

alamirault commented Apr 7, 2023 •
edited
Loading

Mailers transport bridges are listed here: https://symfony.com/doc/5.0/components/mailer.html#transport with DSN examples.

Likes Notifiers (#18119) WDYT to move mailer bridges docs in repo README.md directly ?

For this example we can explain how option message_stream can be used and impacts

(symfony/symfony#42982 is also specific for Amazon Mailer and it will be logic to have doc directly in the readme)

@mdoutreluingne
Copy link
Contributor

I agree with @alamirault, let's take the same logic as the Notifiers (#18119). However, we must target 5.4 with Mailers transport bridges are listed here : https://symfony.com/doc/5.4/mailer.html#using-a-3rd-party-transport

This was referenced Apr 10, 2023
Merged
Merged
@alamirault
Copy link
Contributor

I created #18185 and symfony/symfony#49991.
Let's iterate and discuss on this !

javiereguiluz added a commit that referenced this issue Apr 11, 2023
@javiereguiluz
minor #18185 [Mailer] Link to bridges READMEs (alamirault)
d8c3c22 
This PR was merged into the 5.4 branch.

Discussion
----------

[Mailer] Link to bridges READMEs

#15793 (comment)

PR on symfony/symfony#49991 to document DSN in Readme (some options were missing)

Commits
-------

5d02675 [Mailer] Link to bridges READMEs
fabpot added a commit to symfony/symfony that referenced this issue Apr 11, 2023
@fabpot
minor #49991 [Mailer] Add bridge documentation (alamirault)
026286b 
This PR was merged into the 5.4 branch.

Discussion
----------

[Mailer] Add bridge documentation

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | [From a doc issue](symfony/symfony-docs#15793 (comment))
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!-- required for new features -->

Like work done [for notifiers documentation](#49782). This PR document mailer bridges in their README

(I harmonized examples)

Commits
-------

7ac6fcf [Mailer] Add brifge documentation
@fabpot fabpot closed this as completed Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Keep open Mailer
Projects
None yet
Milestone
5.4
Development

No branches or pull requests
5 participants
@fabpot @xabbuh @alamirault @carsonbot @mdoutreluingne