ah, indeed i was talking about issue #345 :) sorry


On Thu, May 22, 2014 at 1:25 AM, Lakshmi <notifications@github.com> wrote:

> In app/models/student_exercise.rb:
>
> > @@ -38,7 +40,14 @@ class StudentExercise < ActiveRecord::Base
> >
> >    validate :changes_match_state, :on => :update
> >    validate :has_at_least_one_free_response, :on => :update
> > -
> > +
> > +  scope :by_student, lambda { |present_user|
>
> @kjdav <https://github.com/kjdav> - Thanks so much for the thorough
> review and detailed feedback. Much appreciated.
>
> Regarding direct link to the question:
> Are you talking about: #345 <https://github.com/lml/ost/issues/345>? I
> haven't addressed that issue yet. Been focusing only on #340<https://github.com/lml/ost/issues/340>and
> #341 <https://github.com/lml/ost/issues/341>.
> I will fix the rest of the issues you have noted.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/lml/ost/pull/348/files#r12935382>
> .
>