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

Document usage of Symfony Mailer for error email#14360

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
javiereguiluz merged 4 commits intosymfony:5.1fromnoniagriconomie:patch-11
Oct 7, 2020

Conversation

noniagriconomie
Copy link
Contributor

@noniagriconomienoniagriconomie commentedOct 6, 2020•
edited
Loading

Related tosymfony/monolog-bundle#354
andsymfony/monolog-bundle#354 (comment)

Should we also document that for thev4.4 of this page, it will not be possible?
TheSymfony\Bridge\Monolog\Handler\MailerHandler was added inv5.1 (symfony/symfony#33456)

Because the wordingcompatible yet implies that it will be compatible in the future

Thank you

edit: I do not understand theDOCtor-RST check here :s

Error: You are not allowed to use version "3.6.0". Only major version "5" is allowed.Error: Please only provide ".. versionadded::" if the version is greater/equal "5.0"
@noniagriconomie
Copy link
ContributorAuthor

thx@OskarStark :)

just what do you think of the same sentence on the 4.4 code? should we alter the wording?

@javiereguiluzjaviereguiluz added this to the5.1 milestoneOct 7, 2020
@javiereguiluzjaviereguiluz merged commit1cf3cd6 intosymfony:5.1Oct 7, 2020
@javiereguiluz
Copy link
Member

Thanks Antoine! This is now merged in 5.1 and 5.x. You are right about tweaking the message in 4.4 version, so we did that in70c96fb

noniagriconomie reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@OskarStarkOskarStarkOskarStark approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
5.1
Development

Successfully merging this pull request may close these issues.

4 participants
@noniagriconomie@javiereguiluz@OskarStark@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp