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.

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

[Mailer] Add a way to inject Stamps when sending an email via Messenger #17111

Closed
fabpot opened this issue Aug 5, 2022 · 2 comments · Fixed by #17390
Closed

[Mailer] Add a way to inject Stamps when sending an email via Messenger #17111

fabpot opened this issue Aug 5, 2022 · 2 comments · Fixed by #17390
Labels
help wanted Issues and PRs which are looking for volunteers to complete them. Mailer
Milestone
6.2

Comments

@fabpot
Copy link
Member

fabpot commented Aug 5, 2022

Q A
Feature PR symfony/symfony#47191
PR author(s) @fabpot
Merged in 6.2
@xabbuh xabbuh added this to the 6.2 milestone Aug 5, 2022
@wouterj
Copy link
Member

wouterj commented Oct 16, 2022

Hi @fabpot! I'm not sure I can completely follow the changes, whether this removes the MessageEvent or adds an additional event.

I guess we need to add/change https://symfony.com/doc/current/mailer.html#mailer-events Are you able to help us out? :)

@wouterj wouterj added the help wanted Issues and PRs which are looking for volunteers to complete them. label Oct 16, 2022
@fabpot fabpot mentioned this issue Oct 23, 2022
Merged
@fabpot
Copy link
Member Author

fabpot commented Oct 23, 2022

@wouterj See #17390

@fabpot fabpot closed this as completed Oct 23, 2022
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this issue Oct 26, 2022
@javiereguiluz
minor symfony#17390 [Mailer] Add docs about Mailer stamps (fabpot)
9062a32 
This PR was merged into the 6.2 branch.

Discussion
----------

[Mailer] Add docs about Mailer stamps

Closes symfony#17111

Commits
-------

7dbe950 Add docs about Mailer stamps
@MrYamous MrYamous mentioned this issue Jan 12, 2023
Merged
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this issue Jan 13, 2023
@javiereguiluz
minor symfony#17730 [Mailer] remove QueuingMessageEvent (MrYamous)
0846490 
This PR was merged into the 6.2 branch.

Discussion
----------

[Mailer] remove `QueuingMessageEvent`

Related to [47992 in code](symfony/symfony#47992) - QueuingMessageEvent class has been removed due to BC break

Related to symfony#17111 in doc

Remove explanations about this specific event but keep new methods added in previously existing MessageEvent

Commits
-------

9bc26d1 remove QueuingMessageEvent from documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Issues and PRs which are looking for volunteers to complete them. Mailer
Projects
None yet
Milestone
6.2
Development

Successfully merging a pull request may close this issue.

[Mailer] Add docs about Mailer stamps fabpot/symfony-docs
4 participants
@fabpot @wouterj @xabbuh @carsonbot