Jguzaro2018017/entry-challenge#17
Conversation
JonathanL1ma
left a comment
JonathanL1ma
left a comment
There was a problem hiding this comment.
Buen comienzo, soluciona los comentarios de aquí abajo. 😀
| }); | ||
|
|
||
| server.post("/", async (req: Request, res: Response) => { | ||
| server.post("/", async (req: express.Request, res: express.Response) => { |
There was a problem hiding this comment.
No se tiene que modificar nada en este archivo.
| @@ -0,0 +1,31 @@ | |||
| export const data = { | |||
There was a problem hiding this comment.
Deberías de instalar Prettier en tu editor, está te autoidenta tu código y quedará mas legible.
| import { data } from "./data"; | ||
|
|
||
| const axios = require('axios'); | ||
| var contraseña = data.credentials.password; |
There was a problem hiding this comment.
Una buena práctica al momento de programar es escribirlo en inglés, además que escribirlo en español con carácteres especiales como lo es la ñ te podría generar incovenientes.
|
|
||
| } | ||
|
|
||
| async function guetData(){ |
There was a problem hiding this comment.
El nombre de está función contiene un typo, esto quiere decir que posees un error de escritura, la palabra correcta sería get.
|
|
||
|
|
||
|
|
||
|
|
There was a problem hiding this comment.
Cuándo instales Prettier he identes tu código estas líneas innecesarias desaparecerán.
| } | ||
|
|
||
| sendPost(); | ||
| guetData(); |
| async function sendPost(){ | ||
| let res = await axios.post('http://95.217.235.69/', data) | ||
| .then(function(response){ | ||
| console.log(response); |
There was a problem hiding this comment.
No se tienen que dejar console.logs en el código.
| } | ||
| }) | ||
| .then(function(response){ | ||
| console.log(response.data) |
| @@ -0,0 +1,297 @@ | |||
| { | |||
There was a problem hiding this comment.
Estamos utilizando yarn para instalar las dependencias, así que tienes que borrar este archivo, al momento de instalar nuevamente las dependencias con yarn se autogenera un archivo yarn.lock, ese es el que debes de subir.
| username: "jguzaro2018017", | ||
| }, | ||
| credentials: { | ||
| password: "GuzaroTG2482", |
There was a problem hiding this comment.
Está información es privada, por lo que debes de buscar una manera para introducir está información de una manera indirecta.
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
Gracias por tus comentarios @JonathanL1ma
@jguzaro2018017 quedamos en espera de tus actualizaciones.
4 participants
Se modifico la extensión de index.js a index.ts en el archivo package.json
Se creo el archivo client.ts quien era el encargado de realizar las solicitudes POST & GET
Se creo el archivo data.ts en el cual iban los datos a enviar
Se instalaron las dependencias faltantes y se modifico el archivo index.ts