Code review comment for lp:~leonardr/launchpad/temporary-integration

On Wed, 2010-10-20 at 13:04 +0000, Leonard Richardson wrote:
> I'd also appreciate some advice on how to group the action buttons
> into two sets. Should I just put two Actions objects into the context,
> "allow_actions" and "deny_action"?

Why do you want them in two sets?

I think that to render the buttons like I proposed you'll need to do so
manually. You'd still define all your @action methods normally but then
in your template you fill the "buttons" slot and layout the buttons as
you want.