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

endpoint nuevo que regresa toda la lista de explorers filtrados por un stack #174

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

Conversation

jesusbeb
Copy link

Para hacer funcionar el nuevo enpoint de manera local despues de clonar el repositorio, hice los siguientes pasos:

  1. Modifique el archivo package.json en la seccion de scripts, quedando de la siguiente manera:
    "scripts": {
    "start": "node index.js",
    "test": "node --experimental-vm-modules ./node_modules/jest/bin/jest",
    "linter": "node ./node_modules/eslint/bin/eslint.js .",
    "linter-fix": "node ./node_modules/eslint/bin/eslint.js . --fix",
    "server": "node ./lib/server.js"
    },
    ya que uso el Windows 10 y asi como venia, tendria problemas de ejecucion

  2. Instale la dependencia express con npm install express --save

  3. En el archivo ExplorerService agregue un nuevo metodo estatico que recibe dos parametros:
    static getExplorersByStack(explorers,mission)

  4. En el archivo ExplorerController agregue otro metodo que solo recibe un parametro:
    static getExplorersByStack(mission){
    y llama al metodo ExplorerService.getExplorerByStack, le manda el parametro que recibe y el otro se lo envia mediante una linea de codigo que llama a otro metodo para leer el archivo explorers.json

  5. En el archivo server.js creo un endpoint con la url
    localhost:3000/v1/explorers/stack/:mission
    el cual recibe la mision para el stack por query params y muestra la lista de explorers que cumplan la condicion

  6. Corro linter y corrijo errores

  7. Solo hago commit de los archivos que no generaran conflictos al hacer el fork

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