8000 Fixed the way flash messages are retrieved in the template by javiereguiluz · Pull Request #7557 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

Fixed the way flash messages are retrieved in the template #7557

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
Mar 1, 2017

Conversation

javiereguiluz
Copy link
Member

In #7432 I changed the code of the templates to get the flash messages. I don't know why I did that ... because it's completely wrong 😵

In #7555 @lerox pointed this issue and I can confirm it:

symfony_error

So, let's revert it.

@lerox
Copy link
lerox commented Mar 1, 2017

@javiereguiluz 👍

@xabbuh
Copy link
Member
xabbuh commented Mar 1, 2017

Thank you for fixing this so fast Javier and sorry to the inconvinience this caused to everybody.

@xabbuh xabbuh merged commit 8ac43ca into symfony:2.7 Mar 1, 2017
xabbuh added a commit that referenced this pull request Mar 1, 2017
…e (javiereguiluz)

This PR was merged into the 2.7 branch.

Discussion
----------

Fixed the way flash messages are retrieved in the template

In #7432 I changed the code of the templates to get the flash messages. I don't know why I did that ... because it's completely wrong 😵

In #7555 @lerox pointed this issue and I can confirm it:

![symfony_error](https://cloud.githubusercontent.com/assets/73419/23469257/6e423134-fea3-11e6-8d08-e56b01839404.png)

So, let's revert it.

Commits
-------

8ac43ca Fixed the way flash messages are retrieved in the template
@xabbuh
Copy link
Member
xabbuh commented Mar 1, 2017

Don't we have to update PHP template too (see #7559)?

xabbuh added a commit that referenced this pull request Mar 3, 2017
This PR was merged into the 2.7 branch.

Discussion
----------

fix usage of the session flag bag API

completes #7432 and #7557

Commits
-------

f050919 fix usage of the session flag bag API
@javiereguiluz javiereguiluz deleted the revert_7432 branch May 24, 2018 16:05
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
0