Thanks.

Going by Amir's "...keep the scope to the one form we have when
adding/editing ideas." in https://github.com/winnipegrb/winnivote/issues/26,
I'm not gonna adjust anything here for now, as I'm not sure whether there
is any actual intention to merge this in.




On Mon, Feb 10, 2014 at 8:10 PM, Sean Walberg <notifications@github.com>wrote:

> In app/views/devise/shared/_links.slim:
>
> > @@ -0,0 +1,16 @@
> > +- links_to_display = []
>
> If you're going to take it into a helper then you may as well kill the
> partial and have a high level helper that displays the links (which could
> call private methods for each link to avoid a huge method). e.g. where we
> have = render "devise/shared/links" it would become something like =
> links_for_devise.
>
> But I'd still have it dump out a styled ul/li, so you may want to do that
> first and see if anything needs refactoring after that.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/winnipegrb/winnivote/pull/30/files#r9611423>
> .
>