LucasSolares/entry-challenge-v1#6
Conversation
Ktoxcon
left a comment
Ktoxcon
left a comment
There was a problem hiding this comment.
Buen comienzo amigo!
Por favor resuelve los siguientes comentarios.
| const sendSchema = async () => { | ||
| try { | ||
| const response = await Axios.post(process.env.HOST, { | ||
| contactInfo: { |
There was a problem hiding this comment.
Por favor, coloca tu información en un ./src/data.json e importalo para reducir el tamaño de este file y mejorar la organización.
There was a problem hiding this comment.
Mira yo pensaría que es mejor que sea un data.js de este modo podemos seguir usando el process.env sin ningún problema dentro del JS.
There was a problem hiding this comment.
@LucasSolares esa también es buena idea. Pero entonces que sea un ./src/data.ts.
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
Gracias por tus comentarios @Ktoxcon
@LucasSolares por favor resuelve los comentarios adicionales.
| @@ -0,0 +1,1265 @@ | |||
| { | |||
There was a problem hiding this comment.
Estamos usando yarn para las dependencias. Por lo tanto yarn.lock es el archivo autogenerado. Por favor, borra este file.
| import Ajv from "ajv"; | ||
| import { compare, genSalt, hash } from "bcryptjs"; | ||
| import express from "express"; | ||
| import express, { Request, Response } from "express"; |
There was a problem hiding this comment.
Creo que no necesitamos estos imports. Por lo tanto no debe haber cambios en este file.
Puedes revertir los cambios con: git checkout master src/index.ts
Agrega esta configuración en los settings de tu editor:
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
There was a problem hiding this comment.
Excelente la configuración de Vscode.
| ], | ||
| }, | ||
| }); | ||
| const data = (await import('./data')).default |
3 participants
• Install types for express and import Request and Response to the index
• Implements POST Request with the schema
• Add get request and then for await