ufabc-next / ufabc-next-web

📈 Plataforma web para auxiliar na graduação dos alunos da UFABC com informações sobre a performance, processo de matrículas e os discentes da universidade

https://ufabcnext.com

GNU Affero General Public License v3.0

59 stars 18 forks source link

feat/recovery-page #162

Closed mateusbrg closed 10 months ago

mateusbrg commented 11 months ago

Descrição

Página de recuperação de senha do UFABC Next com as modernizações necessárias em Vue 3.

Issues relacionados

158 - Reescrita da página de recovery

Como testar esse PR

Por enquanto, é necessário estar deslogado do Next e colocar '/recovery' na URL para entrar na página
- Caso esteja logado, limpe o seu token em "Inspecionar Elemento > application > localstorage > selecione a URL onde o next está rodando na sua máquina > procure pelo ícone de limpeza, ao lado do 'x' e do campo de filtro e clique nele > reinicie a página.
Verifique que todas as funcionalidades e interações da versão antiga estão funcionando corretamente
Verifique que é possível utilizar as funcionalidades para qualquer tamanho de tela até 360px

Adicionou/atualizou testes automatizados?

[ ] Sim
[ ]Não, porque: Foco em finalizar as implementações e implementar testes durante período de validação no ambiente
[X] Preciso de ajuda para escrever testes

mateusbrg commented 11 months ago

Principais pontos de atenção deste PR até agora:

Falha de segurança nas rotas, consigo abrir qualquer rota pelo URL
acho que faltou um .toLocaleLowerCase() e outras validações no formulário de recuperação de email, o que faz ser necessário, no futuro, a seleção de alguma lib de validação de forms agnóstica de framework

RenanLorijola commented 10 months ago

Principais pontos de atenção deste PR até agora:

Falha de segurança nas rotas, consigo abrir qualquer rota pelo URL

acho que faltou um .toLocaleLowerCase() e outras validações no formulário de recuperação de email, o que faz ser necessário, no futuro, a seleção de alguma lib de validação de forms agnóstica de framework

A segurança das rotas está feita, mas ela não funciona local pois local o app não está em /app e não rodamos a aplicação estática com nenhum server.

Tem algumas libs de validação de schema, mas vou dar uma pesquisada sobre alguma agnostica que ja faça o trabalho todo

RenanLorijola commented 10 months ago

vou aprovar e solucionar o problema das validações com algumas bibliotecas, além disso, vou fazer a logica de só exibir essa página se o usuário estiver deslogado, se estiver logado ele vai parar a páginas de reviews

Joabesv commented 10 months ago

só uns dois cents @RenanLorijola, pra validar dados de maneira agnostica acho que o esquema é Zod mantém a consistencia no backend tbm