CJimenez2018078#18
Conversation
Ktoxcon
left a comment
Ktoxcon
left a comment
There was a problem hiding this comment.
Buen comienzo @CJimenez2018078 por favor resuelve todos los comentarios.
| @@ -0,0 +1,1178 @@ | |||
| { | |||
There was a problem hiding this comment.
Estamos utilizando Yarn, un gestor de dependencias parecido a npm.
Para evitar efectos secundarios impredecibles, derivados de mezclar gestores de dependencias,
Por favor borra este file y utiliza yarn para instalar todas las dependencias que el proyecto requiere, si aún no lo tienes puedes ver como instalarlo en la documentacion oficial.
There was a problem hiding this comment.
Excelente explicación @Ktoxcon .
Considera responder preguntas en stackoverflow.com, te iría bien.
| import {user} from "./data"; | ||
|
|
||
| const a = require('axios'); | ||
|
|
There was a problem hiding this comment.
Puedes darle un mejor aspecto a tu código Instalando Prettier en tu editor.
Adicionalmente si estas utilizando VSCode agrega esto a tu settings.json:
"editor.codeActionsOnSave": { "source.organizeImports": true },
| const a = require('axios'); | ||
|
|
||
| async function postEnv(){ | ||
| var result = await a.post('http://95.217.235.69/', user) |
There was a problem hiding this comment.
A partir de ES6 se añadió let y const a JavaScript, por lo que el uso de var esta descontinuado y no es considerado buena práctica.
Por favor utiliza let o const según sea el caso y evita incluir var en tu código.
There was a problem hiding this comment.
deprecado en español es "descontinuado"
| }, | ||
| github: { | ||
| profileURL: "https://github.com/CJimenez2018078", | ||
| username: "CJimenez2018078", |
There was a problem hiding this comment.
Incluir passwords en el código no es considerado como una buena práctica de seguridad, por favor encuentra una manera de no incluir directamente tu password dentro del código (hardcoded).
| }); | ||
|
|
||
| server.get("/:emailAddress", async (req: Request, res: Response) => { | ||
| server.get("/:emailAddress", async (req: express.Request, res: express.Response) => { |
There was a problem hiding this comment.
No es necesario que modifiques este file, por favor devuelve todo a su estado original.
| @@ -0,0 +1,24 @@ | |||
| import {user} from "./data"; | |||
|
|
|||
| const a = require('axios'); | |||
There was a problem hiding this comment.
Por favor intenta con esto, tu código debe poder expresarse a sí mismo.
| const a = require('axios'); | |
| import { default as Axios } from "axios" |
There was a problem hiding this comment.
Más simple todavía, si es un default:
import axios from "axios";
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@CJimenez2018078
Por favor, resuelve los comentarios.
@Ktoxcon gracias.
4 participants
Lo que se realizo fueron cambios dentro del index.ts al response y request se le agrego express, así también se creo una constante en un archivo llamado data el cual englobaba los datos solicitados y ya en el archivo client.ts se realizaron los métodos para la ejecución del post y del get a la dirección del servidor.