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.

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

Pr/5085 #5430

Closed
wants to merge 4 commits into from
Closed

Pr/5085 #5430

wants to merge 4 commits into from

Conversation

javiereguiluz
Copy link
Member

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets -

schuylr and others added 4 commits March 16, 2015 16:57
There are issues with the current docs as they don't address certain user configurations that prevent Symfony from emailing from their app. Mentioning two factor authentication and the less secure app setting is in my opinion, paramount.

Reference: http://stackoverflow.com/q/29085617/1188035
Revisions concerning the 2-Step-Verification and correct documentation syntax. Thanks @wouterj and @javiereguiluz
@xabbuh
Copy link
Member

xabbuh commented Jun 23, 2015

👍

@weaverryan
Copy link
Member

Thank you Javier and @sjagr!

weaverryan added a commit that referenced this pull request Jul 15, 2015
This PR was submitted for the 2.6 branch but it was merged into the 2.3 branch instead (closes #5430).

Discussion
----------

Pr/5085

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets | -

Commits
-------

bafbd87 Fixed some minor syntax issues.
eb16061 Reflect links correctly in gmail.rst
60703ba Revisions to the gmail.rst doc
0320b5e Update gmail.rst
@weaverryan weaverryan closed this Jul 15, 2015
@javiereguiluz javiereguiluz deleted the pr/5085 branch May 24, 2018 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants