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

Pokedex API #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Pokedex API #1

wants to merge 1 commit into from

Conversation

claunicole
Copy link
Owner

Pokedex API using pokeapi data:
-V1
-Seed database with a Rake task
-CRUD for Pokemon and Types
-Some validations in the models
-Documentation and Readme
-Tests for Pokemon endpoints

@claunicole claunicole changed the title fix and comments Pokedex API Nov 8, 2023
@RodrighoNS
Copy link

Hola Claudia!

Está super buena! instalé y eché a andar la API y todos los endpoints funcionan perfecto ademas de que cumpliste con todos los requerimientos. Te dejo algunos comentarios de cosas que me llamaron la atención:

  1. En el README falta la referencia al comando para correr los test. Primero probé con bundle rspec por costumbre pero luego al revisar /spec me di cuenta que lo hiciste con rails test. Independiente de eso, todo bien con los test 👍.
  2. La carga masiva funciona super bien salvo que solo trae los primeros 21 pokemons. La response contiene paginación como se ve en el pantallazo. De todas formas no te preocupes, es algo que yo no definí bien en los requerimientos así que yo asumo la culpa 😅 además incluir la paginación en la solución que armaste es sencillo y solo extiende el tiempo de procesamiento de la task
    image
  3. El resto está súper bien. Me gustó tu approach con la join table PokemonTypes y las responses de todos los endpoints están con un formato super bueno.

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