ASamayoa/entry-challenge #9
Conversation
There was a problem hiding this comment.
¡Buen comienzo @ASamayoa ! Creo que te estas complicando demasiado, porfavor resolvé los comentarios que te he dejado. Recordá que estamos trabajando con TypeScript 😀.
| @@ -1,5 +1,6 @@ | |||
| .env | |||
|
|
|||
| /public | |||
There was a problem hiding this comment.
No se porqué agregarias la carpeta /public en el .gitignore si está carpeta no existe en el proyecto.
| @@ -5,8 +5,7 @@ | |||
| "main": "index.js", | |||
| "scripts": { | |||
| "test": "echo \"Error: no test specified\" && exit 1", | |||
| "start": "ts-node-dev --no-notify --respawn ./src", | |||
| "start:debug": "ts-node-dev --inspect --no-notify --respawn ./src" | |||
| "start": "nodemon src/index.js" | |||
There was a problem hiding this comment.
No es necesario la utilización de está librería nodemon.
| @@ -15,6 +14,7 @@ | |||
| "axios": "^0.19.2", | |||
| "bcryptjs": "^2.4.3", | |||
| "express": "^4.17.1", | |||
| "nodemon": "^2.0.4", | |||
| { | ||
| "contactInfo": { | ||
| "fullName": "", | ||
| "emailAddress": "" | ||
| }, | ||
| "github": { | ||
| "profileURL": "", | ||
| "username": "" | ||
| }, | ||
| "credentials": { | ||
| "password": "Cambia esta contraseña a una que sólo tú sepas" | ||
| }, | ||
| "personalInfo": { | ||
| "questions": [ | ||
| { | ||
| "question": "If I was a Sr. Programmer, I would like to build:", | ||
| "answer": "Respuesta: ..." | ||
| }, | ||
| { | ||
| "question": "Por favor indica el URL que me lleva a la línea de código de la definición de React.useEffect", | ||
| "answer": "Respuesta: ..." | ||
| }, | ||
| { | ||
| "question": "code is poetry, because:", | ||
| "answer": "Respuesta: ..." | ||
| } | ||
| ] | ||
| } | ||
| } |
There was a problem hiding this comment.
Está información debería de estar en un archivo de data.ts.
| @@ -0,0 +1,114 @@ | |||
| const Ajv = require("ajv"); | |||
There was a problem hiding this comment.
Este archivo index.js debe de ser index.ts y no se tiene que modificar nada en el, las peticiones que haces tienen que ir en un archivo client.ts.
There was a problem hiding this comment.
Regresa al index.ts que se te proporciono al momento de clonar el repositorio.
| @@ -0,0 +1,6 @@ | |||
| const express = require("express"); | |||
There was a problem hiding this comment.
Se tiene que trabajar con TypeScript, regresá este archivo a un server.ts.
| @@ -1,15 +0,0 @@ | |||
| { | |||
There was a problem hiding this comment.
No debes de eliminar este archivo debido a que estamos trabajando con TypeScript.
| @@ -0,0 +1,1875 @@ | |||
| { | |||
There was a problem hiding this comment.
Estamos utilizando yarn para instalar las dependencias, borra este archivo y al momento que instales las dependencias con yarn se te autogenerara un archivo yarn.lock.
There was a problem hiding this comment.
@JonathanL1ma gracias por tus comentarios.
@ASamayoa por favor resuelve las observaciones.
Puedes inspirarte de otros PRs que ya han sido completados:
https://github.com/corecodeio/entry-challenge-v1/pulls
procura entender el funcionamiento y has los cambios pertinentes.
| "questions": [ | ||
| { | ||
| "question": "If I was a Sr. Programmer, I would like to build:", | ||
| "answer": "Respuesta: ..." |
No description provided.