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

Contribución: creando endpoint que regresa lista de explorers filtrados por stack #165

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

MikoVelascoRamirez
Copy link

Contribución Open Source

Se agregó un nuevo endpoint con la url localhost:3000/v1/explorers/stack/javascript que regresará un arreglo con los explorers que tengan javascript dentro del mismo.

Para esta pequeña contribución, se realizó lo siguiente:

  1. Se instalaron las siguientes dependencias necesarias que permitieran darle funcionalidad:
    • Jest
    • Supertest
    • Linter
    • Express

code1

  1. Se designó dentro de las responsabilidades del servicio de /lib/services/ExplorerService.js un método denominado getExplorersByStack(explorers, mission) que regresa dicha lista filtrada por stack. (Tests incluídos en test/services/ExplorerService.test.js para corroborar funcionalidad de código legado).

code2

  1. Se implementó el método de getExplorersByStack(mission) en el controlador lib/controllers/ExplorerController.js extendiendo la funcionalidad de la clase. (Tests incluídos en test/controllers/ExplorerController.test.js) para corroborar funcionalidad de código legado.

code3

  1. Finalmente se implementó el endpoint ya descrito en lib/server.js, regresando un objeto con la tecnología y los explorers resultantes por stack. (Tests incluídos en test/controllers/ExplorerController.test.js) para corroborar funcionalidad de endpoints legados e implementado.

server

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.

1 participant