Skip to content

Conversation

@LuceroLuciano
Copy link

Contribución Open Source

El siguiente código es un ejemplo de como se puede contribuir al código abierto.

  1. Como primer punto se realizo un fork al respositorio, esto crea una copia del código en nuestro repo.

  2. Abrir el repositorio, leer y entender las instrucciones. En este caso se requiere crear un nuevo endpoint que muestre la lista de explorers filtrados por stacks.

Solución

El proceso que se llevo a cabo para crear el nuevo endpoint es el siguiente:

  1. Crear un nuevo método static llamado getListExplorersByStack que recibirá dos parámetros y filtrara la lista de explores utilizando el metodo filter e include para que se muestre los valores del stack. Este método se creara en el archivo ExplorerService.js

Imagen1

  1. Crear la prueba de unidad en el archivo ExplorerService.test.js para verificar que funciona correctamente.
  • Para ejecutar la prueba corre el siguiente comando: npm test test/services/ExplorerService.test.js
    En este paso ya deberías tener instalado jest

Imagen2

  1. Crear un método static en el archivo ExplorerController.js que contiene el método que manda a llamar al método getListExplorersByStack de ExplorerService.js

Imagen3

  1. Finalmente se crea el nuevo endpoint en el archivo server.js
    Imagen4

  2. Ejecutar ESLInt como guía de diseño para corregir los errores de identación, comillas, punto y coma, entre otros.

  • npm run linter para ver los errores
  • npm linter --fix para corregir los errores
  1. Quitar del control de versiones el package-lock.json
  • Primero crear un archivo llamado .gitignore y agregar el **/pakcage-lock.json dentro del archivo .gitignore
  • Ejecutar el siguiente comando git rm --cache package-look.json
  • Aguarda los cambios y haz commit y push

Enpoint en funcionamiento

Lista de explorers filtrada por stacks

En el navegador:

imagen7

Con Postman:
Imagen8

Imagen9

@LuceroLuciano
Copy link
Author

Contribución Open Source

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