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

Updated the code to display flash messages #5431

Closed
wants to merge 2 commits into from

Conversation

javiereguiluz
Copy link
Member

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

aykin and others added 2 commits April 15, 2015 10:27
The sample code does not work due to an array o string conversion exception. The documentation needs to be updated to reflect the change
@xabbuh
Copy link
Member

xabbuh commented Jun 23, 2015

👍

@wouterj
Copy link
Member

wouterj commented Jun 27, 2015

You're doing a great job at filling our notification list Javier! :) Happy to have this bug fixed. Thanks to @aykin as well for the original work!

wouterj added a commit that referenced this pull request Jun 27, 2015
…uiluz)

This PR was submitted for the 2.6 branch but it was merged into the 2.3 branch instead (closes #5431).

Discussion
----------

Updated the code to display flash messages

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

Commits
-------

c2676ae Improved the code used to display flash notices
176cdc6 Update the_controller.rst
@wouterj wouterj closed this Jun 27, 2015
@javiereguiluz javiereguiluz deleted the pr/5181 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.

3 participants