Skip to content

Update Doc : Symfony 4.3 - Swift Mailer #12040

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

Merged
merged 1 commit into from
Jul 26, 2019
Merged

Conversation

TheGarious
Copy link
Contributor

Review Example, erase HelloBundle architecture by templates architecture.

Review Example, erase HelloBundle architecture by templates architecture.
Copy link
Member

@weaverryan weaverryan left a comment

Choose a reason for hiding this comment

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

I didn’t notice any other bad bundle references in the doc.

@TheGarious
Copy link
Contributor Author

In doc Symfony v4.4, this reference of bundle exist.

@javiereguiluz javiereguiluz added this to the 4.3 milestone Jul 26, 2019
@javiereguiluz
Copy link
Member

Gary, thank you very much for fixing this error! Congrats on your first Symfony Docs contribution too!

@javiereguiluz javiereguiluz merged commit f8d783b into symfony:4.3 Jul 26, 2019
javiereguiluz added a commit that referenced this pull request Jul 26, 2019
This PR was merged into the 4.3 branch.

Discussion
----------

Update Doc : Symfony 4.3 - Swift Mailer

Review Example, erase HelloBundle architecture by templates architecture.

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

Commits
-------

f8d783b Update Doc : Symfony 4.3 - Swift Mailer
@javiereguiluz
Copy link
Member

We merged this in "4.4" and "master" branches too. There are other occurrences of "HelloBundle" in the docs ... but all of them look legitimate to me because they are used when talking about bundles. Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants