Skip to content

Sprint 4

Matheus edited this page Nov 1, 2021 · 2 revisions

Participantes

José Aquiles + Gabriel Tiveron

Atividade Descrição
Repositório Api
Issue #3
Pull Request #13
O que foi feito foi testado as rotas de cpu, submission, dataset,task, tpu e login
Problema falta testar as rotas de paper, paper_with_code_integration, model e sota, elas aparentam algumas relações complicadas, os mantenedores não responderam quanto a dividir esta issue em várias issues e avisaram que estão com falta de tempo
Observação Foi comentado sobre a quebra da issue para não refletir em queda de volume de trabalho, visto que é uma issue bem trabalhosa. Essa questão foi levantada que comentários no PR, porém não houve uma resposta por parte dos mantenedores
Solução continuar implementando esta issue

Fellipe Araujo

Atividade Descrição
Repositório computerprogress.com
Issue #44, #48 e #41
Pull Request
  • #57 - este Pull Request é referente a issue #44
  • #58 - este Pull Request é referente a issue #48
O que foi feito
  • No PR #57 foi atualizada algumas bibliotecas javascript para a sua última versão estável.
  • No PR #58 foi corrigida a mensagem de warning "React does not recognize the isHome prop on a DOM element" na página home.
Problemas
  • Para o PR #58, encontrei grandes dificuldades de corrigir a mensagem de warning. A princípio, a solução era simples: um objeto passado como propriedade para um componente deveria estar desestruturado. O problema encontrado foi que o objeto já estava sendo passado de forma desestruturada. E em todos os lugares, inclusive na documentação do React, somente essa solução era dada. Por esse motivo, como estava difícil, depois de 1 semana eu mudei para a issue #44. Ao finalizá-la, retomei para a issue inicial #48 e, depois de muita procura, consegui resolvê-la.
  • Para a issue #41, encontrei alguns problemas de "margin/padding". Aparentemente, a grande maioria das tags html relacionadas a texto possui algum tipo de "margin" ou "padding" que não estão definidas no código. E a simples mudança dessas tags para o Typography do material ui, acaba removendo essas "margin" e "padding".
Soluções
  • Para o PR #58, a solução foi marcar a propriedade do componente como temporária, fazendo com que ela não seja adicionada ao elemento DOM renderizado ou passá-la para baixo na hierarquia do componente.
  • Para a issue #41, ainda não foi encontrada uma solução, ficando, assim, como débito para a próxima sprint.

João

Atividade Descrição
Repositório computerprogress.com
Issue
  • #42 - Set up prettier, eslint and husky
Pull Request
  • #55 - Setup Prettier, ESLint and Husky
O que foi feito
  • PR 55 - Issue #42: As ferramentas ESLint, Prettier e Husky foram configuradas no repositório. Além disso, utilizando a ferramenta Prettier, o código fonte do projeto foi formatado e, utilizando a ferramenta ESLint, os erros de estilo de código foram identificados e corrigidos.
Problema
  • PR 55 - Issue #42: Erros de estilo de código de solução manual e conflitos gerados pela formatação do código.
Solução
  • PR 55 - Issue #42: Os conflitos foram solucionados e foram realizadas pesquisas na documentação do React Hooks.
Clone this wiki locally