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

Appearance settings

[DependencyInjection] Fixed deprecated default message template with XML #19439

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 27, 2016

Conversation

jeremyFreeAgent
Copy link
Contributor

Q A
Branch? 2.8
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

@jeremyFreeAgent jeremyFreeAgent force-pushed the fix/xmlDeprecatedDefaultMessage branch from 3cbb8ca to b7df646 Compare July 26, 2016 16:27
@jeremyFreeAgent jeremyFreeAgent force-pushed the fix/xmlDeprecatedDefaultMessage branch from b7df646 to 165529b Compare July 26, 2016 16:50
@nicolas-grekas
Copy link
Member

👍

@javiereguiluz
Copy link
Member

👍

@stof
Copy link
Member

stof commented Jul 27, 2016

Good catch, thanks @jeremyFreeAgent.

@stof stof merged commit 165529b into symfony:2.8 Jul 27, 2016
stof added a commit that referenced this pull request Jul 27, 2016
…plate with XML (jeremyFreeAgent)

This PR was merged into the 2.8 branch.

Discussion
----------

[DependencyInjection] Fixed deprecated default message template with XML

| Q             | A
| ------------- | ---
| Branch?       | 2.8
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

Commits
-------

165529b [DependencyInjection] Fixed deprecated default message template with XML
This was referenced Jul 30, 2016
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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.