BaseController also handles permitting parameters, which might need to get
factored away somewhere else if you're wanting to make a controller to just
handle authenticated users.

(Think we can use AppController::Base here though.)


On Tue, Feb 11, 2014 at 9:46 AM, Richard D. Bartlett <
notifications@github.com> wrote:

> In app/controllers/translations_controller.rb:
>
> > @@ -0,0 +1,17 @@
> > +class TranslationsController < BaseController
> > +  skip_before_filter :authenticate_user!, only: :translate
>
> This seems like a smell. You might like to inherit from
> ApplicationController so you don't have to skip authentication. Note to
> selves: maybe baseController should be called AuthenticatedUserController...
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/loomio/loomio/pull/1281/files#r9601073>
> .
>