Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
JoaoFelipe
left a comment
JoaoFelipe
left a comment
There was a problem hiding this comment.
Adicionei comentários com sugestões de mudanças. A funcionalidade de upload de xlsx não funcionou pra mim e não consegui identificar o motivo.
| $(document).on('as:action_success', function() { | ||
| $('.class_enrollments-sub-form').each(function() { | ||
| var subform = $(this); | ||
| if (subform.find('[course_has_grade="false"]').length>0){ |
There was a problem hiding this comment.
Acho que seria melhor esconder o campo pelo helper do rails, ao invés de javascript. Tenho a impressão de que ficaria mais consistente com outras configurações semelhantes no projeto e com mais garantia de esconder em todo lugar que venha a incluir subform class_enrollment.
(mas não vejo problema manter aqui, se for complicado fazer por lá)
| if (digits === '') return; | ||
| var number = parseInt(digits, 10); | ||
| var formatted = (number / 10).toFixed(1).replace('.', ','); | ||
| $(this).val(formatted); |
There was a problem hiding this comment.
O cursor na nota está um pouco estranho: se você começa a digitar um valor (exemplo 6,5) e percebe que errou o primeiro dígito, ao apagar o primeiro dígito (por exemplo, pra digitar 7,5), o cursor é movido para o fim (nesse exemplo, a nota fica 5,7).
Imagino que o motivo seja essa linha substituindo o valor pelo valor formatado sem considerar a posição do cursor
| class_enrollment.save | ||
| end | ||
| end | ||
| flash[:info] = "Notas importadas com sucesso!" |
There was a problem hiding this comment.
Olhando o código, não vejo nada errado, mas a funcionalidade de upload de xlsx em turma não funcionou para mim. Aparece a mensagem dizendo que foi carregado, mas as notas não alteraram.
Além disso, poderia ser interessante ter uma tela mostrando as alterações que foram feitas com o upload. Ajudaria tanto nesse debug quanto como feedback pro professor saber que funcionou
| class_enrollment = @course_class.class_enrollments.find_by(enrollment: enrollment) | ||
| if class_enrollment | ||
| class_enrollment.grade = grade | ||
| if class_enrollment.grade.to_i >= minimum_grade_for_approval |
There was a problem hiding this comment.
Nesse if só está considerando a nota para definir Aprovado/Reprovado, mas seria bom considerar Freq S/I para reprovação por falta também.
No caso de reprovação por falta, se a nota final > nota configurada para reprovação por falta, a nota deveria ser alterada e isso deveria ficar evidente na tela sugerida no comentário a seguir.
Se for menor ou igual, mantenha a nota digitada.
| page: true, | ||
| type: :member, | ||
| parameters: { format: :xlsx } | ||
| config.action_links.add "import_grades_xls", |
There was a problem hiding this comment.
Essa ação deveria entrar no ability para permitir que professor use a funcionalidade também. Por enquanto, só está habilitada pra admin
| ns = "http://schemas.openxmlformats.org/spreadsheetml/2006/main" | ||
| xml.xpath("//xmlns:row", "xmlns" => ns).each_with_index do |row, index| | ||
| next if index == 0 | ||
| enrollment_number = row.at_xpath('xmlns:c[@r[starts-with(., "B")]]//xmlns:v', "xmlns" => ns)&.text |
There was a problem hiding this comment.
Será que não tem uma forma mais legível de definir quais são as colunas usadas?
| display: none !important; | ||
| } | ||
| .grade-input{ | ||
| text-align: right; |
There was a problem hiding this comment.
Não sei se o motivo é esse e é detalhe, mas o campo de Nota na tela de Inscrições ficou com o valor ligeiramente cortado:
Acredito que precise adicionar algum padding pra resolver.
3 participants
No description provided.