Uncomplete task

I don't have a preference because I don't know anything :) Let me answer
back using questions.

If we put it in the view will the is_authenticated function be called less
often? Will moving the authentication to the views shorten the code?


On Wed, Sep 3, 2014 at 4:06 PM, Kelly Bucey <notifications@github.com>
wrote:

> In reformedacademy/models.py:
>
> > @@ -236,6 +236,12 @@ def progress_for_user(self, user):
> >
> >          return None
> >
> > +    def uncomplete_task(self, user):
> > +        """Deletes TaskProgress object from database"""
> > +        if user.is_authenticated():
>
> I'm going back and forth on this one. I'm wondering if we should have the
> views check for user authentication instead of the model doing it. Thoughts?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/reformedforum/reformedacademy/pull/18/files#r17080100>
> .
>