Conversation
…In addition, create start_date and end_date to course_class
app/models/enrollment_hold.rb
Outdated
| class_enrollments.each do |class_enrollment| | ||
| course_class = class_enrollment.course_class | ||
| unless course_class.end_date < self.start_date || course_class.start_date > self.end_date | ||
| class_enrollment.delete |
There was a problem hiding this comment.
Me parece estranho ter um delete dentro de um validate. Acho que seria melhor num after_update ou after_create
…, now it's a after_commit function, not a validate function of enrollment hold
…in class_enrolments and class_enrollment_requests
… and other to search for class_enrollments in a period. Cancel_class_enrollments is deleted
|
A validação de class enrollment request incomodando um pouco. Me parece razoável ter essa validação, mas se a matricula vai estar trancada para o período seguinte, não deveria ser possível nem abrir a tela de inscrição de disciplina. O que está acontecendo com o PR é aparecer o link para inscrição, deixar o aluno selecionar disciplinas, mas na hora de enviar, aparece erro da seguinte forma: Acho que pode manter a validação, mas também editar a tela a seguir para avisar algo como "Sua matrícula está trancada. Entre em contato com a secretaria caso queira destrancar para esta inscrição" ou algo assim. |
3 participants
No description provided.