Skip to content

[FrameworkBundle] Fix upgrade guides concerning erroneous removal of assets helper #17450

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
Jan 19, 2016
Merged

[FrameworkBundle] Fix upgrade guides concerning erroneous removal of assets helper #17450

merged 1 commit into from
Jan 19, 2016

Conversation

psrpinto
Copy link
Contributor

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

As mentioned in #17041, the upgrade guides for 2.7 and 3.0 erroneously mention that the templating.helper.assets service is removed in 3.0. #17041 cannot fix these issues since it targets 3.0.

This PR should only be merged after #17041 since it depends on it.

@psrpinto psrpinto changed the title Fix upgrade guides concerning erroneous removal of assets helper [FrameworkBundle] Fix upgrade guides concerning erroneous removal of assets helper Jan 19, 2016
@stof
Copy link
Member

stof commented Jan 19, 2016

👍

1 similar comment
@xabbuh
Copy link
Member

xabbuh commented Jan 19, 2016

👍

@fabpot
Copy link
Member

fabpot commented Jan 19, 2016

Thank you @regularjack.

@fabpot fabpot merged commit 5abac56 into symfony:2.7 Jan 19, 2016
fabpot added a commit that referenced this pull request Jan 19, 2016
…s removal of assets helper (regularjack)

This PR was merged into the 2.7 branch.

Discussion
----------

[FrameworkBundle] Fix upgrade guides concerning erroneous removal of assets helper

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

As mentioned in #17041, the upgrade guides for 2.7 and 3.0 erroneously mention that the `templating.helper.assets` service is removed in 3.0.  #17041 cannot fix these issues since it targets 3.0.

This PR should only be merged after #17041 since it depends on it.

Commits
-------

5abac56 Fix upgrade guides concerning erroneous removal of assets helper
@psrpinto psrpinto deleted the fix/upgrade-guides branch July 24, 2016 20:45
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.

5 participants