Closed dduartee closed 3 years ago
Não sabia que tinha a possibilidade de ter mais de uma tabela :) Podemos trocar o if sim! Você poderia abrir um pull request, trocando o if para < 1?
Eu geralmente coloco a regra mais rigida, pois eu tento evitar que ele analize um pagina de formato insperado pensando que está correto. Por exemplo, neste caso, se tivesse uma outra tabela no topo, ele poderia achar que essa tabela é a que contem as notas e retornaria algo errado, assim ele retorna um erro pois ele não garante que vai ser algo correto, mas acho não tem problema mudar este if.
Não sabia que tinha a possibilidade de ter mais de uma tabela :)
Eu também não sabia, levei um susto quando percebi que tinha 2 tabelas.
Você poderia abrir um pull request, trocando o if para < 1?
Claro, abri um pull request, o meu primeiro no caso, qualquer auxilio de como melhorar a pull, como por exemplo: o nome da branch, se devo explicar o motivo do pull...
Aceitei o pull request!
Claro, abri um pull request, o meu primeiro no caso, qualquer auxilio de como melhorar a pull, como por exemplo: o nome da branch, se devo explicar o motivo do pull...
O nome da branch não tem tanta importância desde que o pull request explique, e como a sua explicação já está perfeita, está tudo ótimo. :)
Beleza! valeu pelo aprendizado, vou fechar o issue
Olá, eu estava com um problema ao usar o getGrades em uma matéria, recebendo o erro:
src/courses/sigaa-course-student.ts
Dizendo que a tabela estava vazia. Mas pelo o contrario, tinha 2 tabelas.
Uma para as notas, e uma em relação a recuperação da nota.
Gostaria de saber sobre o
if (table.length !== 1)
, porque!== 1
e não< 1
. Fiz os testes aqui e substitui a condição, retornou normalmente a tabela das notas. Teria algum motivo em especifico para essa condição?