Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[MonologBridge] Add Mailer handler#33456

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
fabpot merged 1 commit intosymfony:masterfromBoShurik:monolog-mailer-handler
Feb 8, 2020
Merged

[MonologBridge] Add Mailer handler#33456

fabpot merged 1 commit intosymfony:masterfromBoShurik:monolog-mailer-handler
Feb 8, 2020

Conversation

BoShurik
Copy link
Contributor

@BoShurikBoShurik commentedSep 4, 2019
edited
Loading

QA
Branch?4.4
Bug fix?no
New feature?yes
BC breaks?no
Deprecations?no
Tests pass?yes
Fixed tickets#33209
LicenseMIT
Doc PR-

acasademont and Chris53897 reacted with hooray emoji
Copy link
Member

@stofstof left a comment

Choose a reason for hiding this comment

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

Tests are missing

@BoShurik
Copy link
ContributorAuthor

@stof done, but not sure why build fails. Looks like unrelated

@nicolas-grekasnicolas-grekas added this to thenext milestoneSep 6, 2019
@weaverryanweaverryan modified the milestones:next,4.4Oct 28, 2019
@weaverryan
Copy link
Member

Proposing this for the 4.4 milestone so that this doesn't block Mailer as a first-class citizen in 4.4/5.0

@nicolas-grekasnicolas-grekas changed the titleAdd monolog mailer handler[MonologBridge] Add Mailer handlerNov 5, 2019
@weaverryanweaverryan modified the milestones:4.4,nextDec 2, 2019
@acasademont
Copy link
Contributor

Any news here? I guess a PR to monolog-bundle would be also needed?

OskarStark added a commit to symfony/symfony-docs that referenced this pull requestDec 17, 2019
…nolog (javiereguiluz)This PR was merged into the 4.3 branch.Discussion----------[Monolog] Add a caution note about Symfony Mailer and MonologWe can'tfix#12758 properly until this is fixed insymfony/symfony#33456  but meanwhile we can display a caution message to avoid confusion.Commits-------82119af [Monolog] Add a caution note about Symfony Mailer and Monolog
@mmarton
Copy link

Any update here? It's the last reason we are still on swiftmailer. :/

ThomasTr and BOB41K1987 reacted with thumbs up emoji

@noniagriconomie
Copy link
Contributor

Hi@BoShurik, how can I help to get this for 5.1?

Thank you,

@BoShurik
Copy link
ContributorAuthor

@noniagriconomie Hi! Sorry for delay. I'll add changes by the end of the week

noniagriconomie reacted with thumbs up emoji

@BoShurikBoShurik changed the base branch from4.4 tomasterJanuary 16, 2020 21:05
@BoShurik
Copy link
ContributorAuthor

Needs a review

@fabpot
Copy link
Member

Thank you@BoShurik.

noniagriconomie reacted with hooray emojiBoShurik, a-menshchikov, and Chris53897 reacted with heart emoji

fabpot added a commit that referenced this pull requestFeb 8, 2020
This PR was merged into the 5.1-dev branch.Discussion----------[MonologBridge] Add Mailer handler| Q             | A| ------------- | ---| Branch?       | 4.4| Bug fix?      | no| New feature?  | yes <!-- please update src/**/CHANGELOG.md files -->| BC breaks?    | no     <!-- seehttps://symfony.com/bc -->| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->| Fixed tickets |#33209   <!-- #-prefixed issue number(s), if any -->| License       | MIT| Doc PR        | - <!-- required for new features --><!--Replace this notice by a short README for your feature/bugfix. This will help peopleunderstand your PR and can be used as a start for the documentation.Additionally (seehttps://symfony.com/roadmap): - 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 4.4. - Legacy code removals go to the master branch.-->Commits-------5b7393b Add monolog mailer handler
@fabpotfabpot merged commit5b7393b intosymfony:masterFeb 8, 2020
@nicolas-grekasnicolas-grekas modified the milestones:next,5.1May 4, 2020
@fabpotfabpot mentioned this pull requestMay 5, 2020
@Schyzophrenic
Copy link

As many others, this is the last reason we use SwiftMailer. Is there any chance this will be merged into SF 4.4?

webdevilopers and gmuehl reacted with rocket emoji

@fabpot
Copy link
Member

@Schyzophrenic No, new features are never backported.

lyrixx added a commit to symfony/monolog-bundle that referenced this pull requestOct 2, 2020
This PR was squashed before being merged into the 3.x-dev branch.Discussion----------Add Symfony Mailer supportAdd support for the new symfony mailer directly on the bundle after the MailerHandle was added to the bridge onsymfony/symfony#33456Also this can makesymfony/symfony-docs#13077 move forwardTests are failing due to incompatible phpunit versions, not sure how to fix this...Commits-------4bea89c Add Symfony Mailer support
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this pull requestOct 7, 2020
…(noniagriconomie, OskarStark)This PR was merged into the 5.1 branch.Discussion----------Document usage of Symfony Mailer for error emailRelated tosymfony/monolog-bundle#354andsymfony/monolog-bundle#354 (comment)----------Should we also document that for the `v4.4` of this page, it will not be possible?The `Symfony\Bridge\Monolog\Handler\MailerHandler` was added in `v5.1` (symfony/symfony#33456)Because the wording `compatible yet` implies that it will be compatible in the futureThank youedit: I do not understand the `DOCtor-RST` check here :sCommits-------202b9e6 Update messaged20f07c Fix: DOXtor-RST922932f Makes DOCtor-RST happy3df4d9e Document usage of Symfony Mailer for error email
@faizanakram99
Copy link
Contributor

How to use MailerHandler without framework ?

dani-danigm pushed a commit to dani-danigm/monolog-bundle that referenced this pull requestJun 15, 2022
This PR was squashed before being merged into the 3.x-dev branch.Discussion----------Add Symfony Mailer supportAdd support for the new symfony mailer directly on the bundle after the MailerHandle was added to the bridge onsymfony/symfony#33456Also this can makesymfony/symfony-docs#13077 move forwardTests are failing due to incompatible phpunit versions, not sure how to fix this...Commits-------0b832f2 Add Symfony Mailer support
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@noniagriconomienoniagriconomienoniagriconomie left review comments

@fabpotfabpotfabpot approved these changes

@stofstofAwaiting requested review from stof

@weaverryanweaverryanAwaiting requested review from weaverryan

Assignees
No one assigned
Projects
None yet
Milestone
5.1
Development

Successfully merging this pull request may close these issues.

11 participants
@BoShurik@weaverryan@acasademont@mmarton@noniagriconomie@fabpot@Schyzophrenic@faizanakram99@stof@nicolas-grekas@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp