yes, def necessary even with resources. it makes it clear to the reader
that this is not an endpoint


On Tue, Jun 3, 2014 at 4:35 PM, Maximilian Tagher <notifications@github.com>
wrote:

> In app/controllers/hacks_controller.rb:
>
> > @@ -0,0 +1,37 @@
> > +class HacksController < ApplicationController
> > +  before_filter :find_hackday
> > +
> > +  def index
> > +    @hacks = @hackday.hacks.all
> > +  end
> > +
> > +  def find_hackday
>
> Do you think this is necessary given that b/c he's using resources, rails
> won't just let this method be called from someone making an HTTP request?
> Probably still good organization to separate it from the rest of the routes
> though
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Heyzap/hackvote-rahul/pull/1/files#r13365144>.
>



-- 
Co-founder, Heyzap.com

My Blog: http://immad.me
Twitter: twitter.com/immad