nova funcionalidade: enviar trabalho em atraso#61
nova funcionalidade: enviar trabalho em atraso#61helderufc wants to merge 13 commits intodevelopmentfrom
Conversation
-erro: situation sent_late é gerada dinamicamente
info retorna agora ... file_sent_late. model assignment e score agora usam esse dado para retorna a situação não foi necessário criar coluna para avisar se foi enviado em atraso
| def can_destroy? | ||
| raise CanCan::AccessDenied unless user_id == User.current.try(:id) | ||
| raise 'date_range_expired' unless assignment.in_time? | ||
| #raise 'date_range_expired' unless assignment.in_time? |
There was a problem hiding this comment.
Essa exceção ficou comentada. Eu acho que para que ela não impeça a exclusão de arquivos enviados em atraso talvez o ideal seja fazer algumas mudanças. Por exemplo, ao invés de verificar se o prazo se encerrou, verificar se o trabalho já foi avaliado (importante para que o aluno não exclua o arquivo enviado em atraso depois que o professor já corrigiu) e seguir talvez com a verificação do prazo encerrado nos casos em que não é permitido envio em atraso.
| end_hour: à | ||
| environment_controlled: 'Você pode controlar o local de onde o aluno irá realizar o trabalho (por exemplo: laboratórios, blocos, etc)' | ||
| environment_not_controlled: 'O aluno poderá realizar este trabalho em qualquer local (por exemplo: laboratórios, casa, lan houses, etc)' | ||
| late_submissions_allowed: "Envios atrasados permitidos" |
There was a problem hiding this comment.
Eu acho que esse texto poderia ser um pouco mais claro. Por exemplo, "Permite que o aluno envie o trabalho após o prazo final" ou algo nesse sentido. Isso vale para o de baixo também.
não é permitido que o aluno envie ou apague um arquivo de um trabalho ja avaliado
There was a problem hiding this comment.
Aqui você tirou a formatação do file_sent_date e as datas acabaram ficando assim:
Antes
Depois
Também é interessante deixar o ' - ' no final para que o campo não fique completamente vazio caso o arquivo não seja enviado. Passa uma informação mais clara para o usuário
There was a problem hiding this comment.
Acredito que precise ter uma melhor apresentação dentro do menu "Acompanhamento" quando logado como professor. Ex: a sigla na tabela e a legenda descrevendo o que a sigla quer dizer.
Também percebi que a modificação pode afetar o valor que aparece nos relatórios, não é um problema em si, somente a cor está divergente.
3 participants
adicionada opção de marcar trabalho enviavel como atraso na hora de criar
quando o aluno envia em atraso, aparece no status em amarelo a situação
contém novas migrations