Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Added flash to the locals w/connect-flash #2

Merged
merged 1 commit into from Aug 29, 2012

Conversation

Projects
None yet
2 participants
Contributor

scottkf commented Aug 21, 2012

I guess you didn't really need connect-flash, but it was easier.

As an aside, do you plan on writing something like this for flatiron?

Owner

pksunkara commented Aug 21, 2012

lgtm, but what about multiple flash messages?

Yes, I plan something like this for flatiron too. This module already uses director, nconf, resourceful from flatiron.

Contributor

scottkf commented Aug 22, 2012

I added support for multiple flash messages, I look forward to trying out flatiron more, the RVP proposal was interesting.

@pksunkara pksunkara added a commit that referenced this pull request Aug 29, 2012

@pksunkara pksunkara Merge pull request #2 from scottkf/master
Added flash to the locals w/connect-flash
660400c

@pksunkara pksunkara merged commit 660400c into pksunkara:master Aug 29, 2012

Owner

pksunkara commented Aug 29, 2012

Done.

@scottkf Thanks. Do you have any more ideas or suggestions?

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