Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removing Stage #45

Merged
merged 35 commits into from
Mar 7, 2023
Merged

Removing Stage #45

merged 35 commits into from
Mar 7, 2023

Conversation

davi-o-barbosa
Copy link
Collaborator

Esse PR foi testado manualmente apenas, porém testei todo o workflow do aplicativo através de requests pelo Hoppscotch.

  • Closes Deleção do modelo stages #36 - Troquei todos os schemas e arrumei as funções dos services. Agora progress.box dentro do usuário é uma array e foi criada outra array no campo progress.available para armazenar quais módulos estão disponíveis, seguindo a proposta da Modificação da lógica de aprovação #38.
  • Closes Modificação da lógica de aprovação #38 - Isso foi implementado no código, agora o próximo módulo é liberado independente do resultado e a resposta da API é apenas a % de respostas SIM, com um número entre 0 e 1. Porém seguindo os requisitos em Thesis changes #34, a lógica de aprovação ainda existe no fundo pra definir se será utilizada a caixa alternativa ou não, com o campo progress.box.$.attempt.
  • Closes Refactor fields #12 - Mudei o que restava de comorbidity para disability.
  • Closes Fix AdminJS panel to users. #33 - Acontece que o erro acontecia pois o AdminJS não aceita o schema Map do MongoDB, então para arrumar eu mudei toda a lógica para utilização de arrays.

@davi-o-barbosa davi-o-barbosa added the high-prior High priority task label Feb 16, 2023
@davi-o-barbosa davi-o-barbosa self-assigned this Feb 16, 2023
Copy link
Collaborator

@nickolasrm nickolasrm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gostei bastante! Deixei alguns comentários para você avaliar e consertar, mas como nada será bloqueante vou deixar aprovado pois a funcionalidade está bem clara para mim.

Uma dica: Já coloca docstring nos métodos q vc ta fazendo pra evitar debito técnico

server/src/model/activity.ts Show resolved Hide resolved
server/src/model/module.ts Show resolved Hide resolved
server/src/route/game/user.ts Outdated Show resolved Hide resolved
server/src/service/module.ts Show resolved Hide resolved
server/src/service/utils/linkedlist.ts Outdated Show resolved Hide resolved
server/src/service/utils/linkedlist.ts Show resolved Hide resolved
@davi-o-barbosa
Copy link
Collaborator Author

davi-o-barbosa commented Mar 3, 2023

Closes #43 - next foi removido e agora utilizamos previous. A abstração de "game" também foi removida, não existindo mais um "pai" para a linked list. Um método para ordernar a lista foi adicionado para garantir a funcionalidade durante a inserção de novos itens.

#33 - Eu mudei as coisas de volta para um Map, isso causa o erro no AdminJS novamente, por isso removi Users do AdminJS, como não é necessário no momento. Caso seja necessário, eu posso mudar o tipo do schema para Mixed em vez de Map, isso faz o Mongoose retornar um objeto, no qual posso fazer parse no servidor para Map porém ainda ser tratado corretamente no painel de adminstrador.

Relacionado a #25 - fc542d2 muda o flow para se alinhar com a nova diagramação que fiz. Realizei mudanças no serviço do usuário e criei a rota necessária para retornar os módulos junto com sua disponibilidade.

@nickolasrm
Copy link
Collaborator

Closes #43 - next foi removido e agora utilizamos previous. A abstração de "game" também foi removida, não existindo mais um "pai" para a linked list. Um método para ordernar a lista foi adicionado para garantir a funcionalidade durante a inserção de novos itens.

#33 - Eu mudei as coisas de volta para um Map, isso causa o erro no AdminJS novamente, por isso removi Users do AdminJS, como não é necessário no momento. Caso seja necessário, eu posso mudar o tipo do schema para Mixed em vez de Map, isso faz o Mongoose retornar um objeto, no qual posso fazer parse no servidor para Map porém ainda ser tratado corretamente no painel de adminstrador.

Relacionado a #25 - fc542d2 muda o flow para se alinhar com a nova diagramação que fiz. Realizei mudanças no serviço do usuário e criei a rota necessária para retornar os módulos junto com sua disponibilidade.

Pode colocar um CHANGELOG.md caso queira ir documentando as maiores mudanças feitas no código

@davi-o-barbosa davi-o-barbosa merged commit 8654294 into dev/api Mar 7, 2023
@davi-o-barbosa davi-o-barbosa deleted the feat/api/removing-stage branch March 14, 2023 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high-prior High priority task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants