search

insper-education / active-handout

https://insper-education.github.io/active-handout/
11 stars 4 forks source link

Report #10

Closed rafaelcorsi closed 2 years ago

rafaelcorsi commented 3 years ago

Adicionado report para firebase #18

rafaelcorsi commented 3 years ago

@igordsm consegue revisar? Ainda precisamos descobrir como fazer com o lance da chave do firebase não ficar pública.

rafaelcorsi commented 3 years ago

@toshikurauchi tomei vergonha na cara e validei todos os módulos com todas as općoes (com e sem report), peguei um erro no quiz e short question pq eu não tratava a resposta do report para saber se o aluno tinha se identificado.

rafaelcorsi commented 3 years ago

Uma questão que apareceu é a seguinte (report = true, identified = true), aluno ainda não se identificou.

O que fazer? Atualmente eu bloqueio a exibićão da resposta correta e não salvo a resposta no localStorage, mas ainda não tem nenhuma forma de avisar o aluno que ele precisa se identificar.

Pensei em substituir o box da resposta correta por um de warning com uma mensagem.

O mesmo se aplica ao quiz

igordsm commented 3 years ago

Acho que esse é um ponto importante: o aluno é obrigado a se identificar para ter a resposta? Queremos que isso seja obrigatório ou a identificação é opcional?

Poderíamos criar alguma maneira de receber os dados anonimizados? Por exemplo, a cada browser usado ele recebe um UUID que é o mesmo durante todo o semestre, mas não o identifica diretamente.

On Sat, May 22, 2021 at 1:37 PM Rafael Corsi @.***> wrote:

Uma questão que apareceu é a seguinte (report = true, identified = true), aluno ainda não se identificou.

  • Preenche short question
  • Clica no botão
  • Aluno não insere o nome ou aperta cancel no popup

O que fazer? Atualmente eu bloqueio a exibićão da resposta correta e não salvo a resposta no localStorage, mas ainda não tem nenhuma forma de avisar o aluno que ele precisa se identificar.

Pensei em substituir o box da resposta correta por um de warning com uma mensagem.

O mesmo se aplica ao quiz

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/insper-education/active-handout/pull/10#issuecomment-846432900, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABWCBVOFEHHLFL3YRNVITLTO7MUXANCNFSM435XV55A .

-- Igor Montagner http://igormontagner.blogspot.com/

rafaelcorsi commented 3 years ago

Oi @igordsm se no arquivo de config tiver: identified: false: eu gero um id automático, mas no caso de estar configurado para true o aluno deve se identificar, porém o login que ele informa não é validado em lugar nenhum.

O problema do id randomizado é vai gerar um para cada site que usa o active-handout e não vamos conseguir rastrear o aluno em cursos.

Podemos explorar outras formas mais completas de autenticação (OAuth, ...)