falcao-g/processo-seletivo-LEDES

Fix: Situações na edição do usuário

Closed this issue · 3 comments

Na edição do usuário, não está sendo incluído o campo "situation" como "ANALYSIS" e também não está sendo verificado se o campo "situation" está como "APPROVED" ou "DISAPPROVED", que são as únicas duas situações que podem ser editadas.

Arquivos para alteração :

  • src/user/user.controller.js
  • src/database/user.js

não foi especificado em momento algum que só é possível editar o usuário quando situation é "APPROVED" ou "DISAPPROVED", se um usuário muda suas informações enquanto seu crachá ainda está em análise, as informações vão continuar em análise

acho que a issue aqui seria que o campo situation do usuário não está sendo setado como "ANALYSIS" depois da edição

Na realidade, ambos os cenários são possíveis. Pode acontecer de o usuário editar uma informação que está sendo analisada pelo administrador, resultando em inconsistências.Ou seja, o administrador pode aprovar um cadastro que o usuário editou momentos antes e ele continua visualizado um cadastro antigo(desatualizado).
E a segunda opção campo situation do usuário não está sendo setado como "ANALYSIS" depois da edição.

verdade, seu contraponto é justo, em resumo temos dois problemas aqui:

  • usuários em análise ainda podem alterar informações
  • usuários após alterar informações não vão para análise