Lo que hice ahí fue asociarle a los admins o docentes que se creen todos
los cursos existentes (para que puedan acceder a cursos históricos).
Si la cuenta en cambio es de un alumno solamente se le asigna el curso
activo.

Pensandolo ahora creo que podría haber problemas para alumnos que esten
recursando la materia, aunque espero que sean pocos y podamos manejarlos
manualmente...


On Tue, Feb 25, 2014 at 5:26 PM, DiegoS <notifications@github.com> wrote:

> In models/account.rb:
>
> >    validates_with_method :tag, :method => :is_valid_tag?
> >
> > +  after :create do
> > +    if self.role == ADMIN || self.role == TEACHER
> > +      # Associate all Courses with Teacher / Admin
> > +      Course.all.each do |c|
> > +        self.courses << c
> > +      end
> > +    elsif Course.active
> > +      self.courses << Course.active
>
> Estaba viendo el PR que mando DiegoM, y para mí esta linea estaría
> haciendo que un *Account* tenga varios cursos. Eso estaría bien en caso
> de que el rol de *Account* fuera *ADMIN* o *TEACHER*.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/fiuba/alfred/pull/83/files#r10054086>
> .
>