areyes2018391(fixed)#5
Conversation
Key file was deleted
Package-lock.jason was deleted
Ktoxcon
left a comment
Ktoxcon
left a comment
There was a problem hiding this comment.
Muy Buen comienzo @areyes2018391
Por favor resuelve los siguientes comentarios.
| .env | ||
|
|
||
| /node_modules | ||
| /dist |
There was a problem hiding this comment.
Solo una única modificación era necesaria en este archivo, por favor devuelve su estado original al archivo y realizala
There was a problem hiding this comment.
Ya he modificado el .gitignore
| method: 'GET', | ||
| headers: { | ||
| 'Content-Type': 'application/json', | ||
| "x-password": "Reyesg2000" |
There was a problem hiding this comment.
Busca una manera de agregar la contraseña sin ponerla directamente en el código (hardcoded).
There was a problem hiding this comment.
La contraseña ha sido removida y ahora uso variables de entorno
| }, | ||
| response => { | ||
| console.log(response.statusCode); | ||
| response.pipe(createWriteStream('./key.txt')) |
There was a problem hiding this comment.
No es necesario que guardes este file.
|
|
||
|
|
||
|
|
||
| const data = { |
There was a problem hiding this comment.
Por favor mueve este bloque a un ./src/data.json o ./src/data.ts , e importalo para reducir el tamaño de este file y mejorar la organización.
There was a problem hiding this comment.
He creado un archivo data.ts y he movido este bloque
There was a problem hiding this comment.
@Ktoxcon cuando refieras nombres de archivos, procura marcarlos como ./src/data.json, ./src/data.ts.
| { | ||
| 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:https://github.com/facebook/react/blob/master/packages/react/src/ReactHooks.js#L104", |
There was a problem hiding this comment.
Estas muy cerca!
Pero aquí están usando un dispatcher, que es como un adminstrador de las acciones.
Por ejemplo, useEffect acepta un parámetro [] que detecta el cambio del contenido. En dónde podría estar esta lógica?
There was a problem hiding this comment.
He encontado el link y he subido los cambios
| console.log("listening"); | ||
| }); | ||
|
|
||
| sendInformation(); |
There was a problem hiding this comment.
No es necesario que modifiques este archivo, llama a tus funciones desde ./src/client.ts
There was a problem hiding this comment.
El index ha sido regresado su estado original
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
Muy cerca.
Gracias @Ktoxcon por las observaciones.
@areyes2018391 por favor instala esta extensión en tu editor: https://github.com/prettier/prettier-vscode
También añade esto a settings,.json de tu editor:
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
| 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.
No es necesario modificar ./src/index.ts.
Ejecuta este comando para devolverlo al estado original: git checkout master src/index.ts
4 participants
No description provided.