Skip to content

Pokemon Go#67

Open
FabioAsada wants to merge 1 commit intomasterfrom
pokemon-go
Open

Pokemon Go#67
FabioAsada wants to merge 1 commit intomasterfrom
pokemon-go

Conversation

@FabioAsada
Copy link
Collaborator

@FabioAsada FabioAsada commented May 28, 2022

O que funciona

  • listagem
  • busca

O que não funciona

  • paginação
  • filtros

Postman

https://documenter.getpostman.com/view/19296247/Uz5CLHcT

@FabioAsada FabioAsada requested a review from bellacbs May 28, 2022 20:25
Copy link

@labenu-bot labenu-bot left a comment

Choose a reason for hiding this comment

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

Parabéns pela entrega!

Requisitos do projeto ✅

Implementações Feito
Criou algum endpoint que faz a listagens de dados dos Pokemons.
Criou algum endpoint que tem a função de pesquisar os dados dos pokemons.
Criou um Readme/documentação para o projeto. -
Fez o deploy do projeto. -
Feedback do código Feito
Importou os dados do Pokemon Go que estão no arquivo 'Pokemon Go.xlsx' e persistiu esses dados em um Banco de dados: MySQL, SQL Server, MongoDB...
Criou uma API seguindo os padrões de arquitetura Restful
Criou algum endpoint que tem a função de paginação. -
Criou algum endpoint que tem a função de filtrar os dados dos pokemons.
Usou NodeJS para criar a aplicação.

Comentários da pessoa avaliadora

Eei Fábio, parabéns pela entrega, esforço e dedicação

  • evite criar enpoints apenas com “/”, como são dados relacionados a pokenons, você pode utilizar ex: “/pokemons/create”, “/pokemons/seach”. Fica um código mais limpo
  • antes de fazer uma operação, como transformar o que for recebido em letra minuscula, verifique primeiro se a informação que você queria foi enviada. Ex: no endpoint “/name”, já é convertido o que receberia do body em letras minusculas, assim, caso o usuário não envie o body, dará o erro Cannot read properties of undefined (reading 'toLowerCase') então seria interessante transformar para letras minusculas, após verificar se foi enviado ou não
  • para os próximos projetos, tente utilizar classes, dividindo em controller, business e data
  • tente também fazer o deploy e readme do projeto para colocar no seu portfólio

Análise realizada por: Izabella Silva

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