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

feat: reviews page migration #145

Merged
merged 13 commits into from
Oct 27, 2023

Conversation

FusiDaniel
Copy link
Collaborator

Descrição

Recria a página de Reviews utilizando vue 3, trazendo todos os dados das APIs do backend, buscando manter a experiência de uso da página antiga, mas com melhorias de interface e responsividade. Foram implementadas a pesquisa e listagem de professor e disciplina, avaliação de professor, e implementada a avaliação de disciplina na página de histórico #131.

Além disso adicionei algumas funções de utilidades, serviço e tipagens

Como testar esse PR

  1. Abra o next em produção com seu login
  2. Abra essa aba em produção com seu token mockado
  3. Verifique que todas as funcionalidades e interações da versão antiga estão funcionando corretamente
  4. Verifique que é possível utilizar as funcionalidades para qualquer tamanho de tela até 360px

Adicionou/atualizou testes automatizados?

  • Sim
  • Não, porque: Pelo grande escopo do PR, e pela necessidade de partes de código aqui presente em outras páginas, vou fazer o testes em outro PR
  • Preciso de ajuda para escrever testes

@FusiDaniel FusiDaniel changed the base branch from master to feat/microfrontend October 6, 2023 16:25
@FusiDaniel FusiDaniel force-pushed the feat/microfrontend-reviews-page branch from 8ef1064 to 2addd92 Compare October 6, 2023 16:28
@RenanLorijola
Copy link
Collaborator

Precisa resolver os conflitos de código

Copy link
Collaborator

Choose a reason for hiding this comment

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

o que acha de colocar num arquivo date esse metodo e também o transformDateTotimeAgo?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Esse aqui n teve typo, é Data mesmo heuheuhe

@RenanLorijola
Copy link
Collaborator

RenanLorijola commented Oct 18, 2023

faz sentido subir esse PR e em seguida trabalhar em um outro de testes, assim que resolver as sugestões deste, pois foi feito bastante coisa que deve ser testada. Se o PR não estivesse tão grande, iria comentar para fazer nele mesmo.

@FusiDaniel FusiDaniel force-pushed the feat/microfrontend-reviews-page branch from 2addd92 to ad787b3 Compare October 27, 2023 03:12
@FusiDaniel FusiDaniel merged commit 7776d5c into feat/microfrontend Oct 27, 2023
2 checks passed
@brMonteiro-G brMonteiro-G deleted the feat/microfrontend-reviews-page branch November 11, 2023 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants