Templating support for flash about tg2

[amol-]

I think it's a good feature, but not one that should involve too much changes, we should try to avoid adding something too complex for a feature that the user can implement by himself in a bunch of lines of code.

the tg.flash.TGFlash class can probably be modified to use a template instead of plain string substitution and also make possible to pass a dictionary as the message instead of a string. Would that be a good enough solution? In that case we can probably manage to make something that is fully backward compatible but also makes possible to render templates inside the flash.

[moschlar]

I'm playing around here: https://github.com/moschlar/tgext.flash

[amol-]

The workshop branch has a different implementation that doesn't rely anymore on webflash and supports python3, if you want to experiment there we can easily merge a proposed patch that keeps backward compatibility.

https://github.com/TurboGears/tg2/blob/workshop/tg/flash.py

[moschlar]

Should be TG>=2.3 compatible in https://github.com/moschlar/tgext.flash/tree/38887b6b084c3e10cd58283c4d071c9941162ab3
I only use imports which are present in <2.3 and >=2.3 .

Maybe we can merge it after some more refinements, I'll tell the mailing list about it, maybe someone has ideas ;).

I'd like to keep it available as an extension so it is usable for <2.3, too!

[amol-]

This is something which looks great to have but is missing some core features. Can we extend tgext.flash with the missing JS support (which is foundamental when used with caching) or should we provide a custom implementation in TurboGears itself?

[moschlar]

See also #52

[amol-]

Solved by 07fc5e2 documented at http://turbogears.readthedocs.org/en/development/turbogears/webflash.html#flash-options

[moschlar]

👍