solicitud#15
Conversation
This reverts commit d581de0.
| } | ||
|
|
||
| postData(); | ||
| var s = 0 No newline at end of file |
There was a problem hiding this comment.
te sugiero que elimines variables innecesarias y que ejecutes la función de getData() dentro de este archivo
| var s = 0 | |
| getData() |
| 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.
las importaciones de Request y Response son innecesarias, tienes que regresar este archivo a su estado original, para esto intenta hacer un git checkout master src/index.ts.
| import express, { Request, Response } from "express"; | |
| import express from "express"; |
|
|
||
| function getData() { | ||
| axios.get(`http://95.217.235.69/${process.env.EMAIL}`,{ | ||
|
|
There was a problem hiding this comment.
te recomiendo instalar la extensión prettier en tu editor para aplicarlo en los archivos de client.ts y data.json
en el archivo settings.json de tu editor agrega lo siguiente:
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
danbart
left a comment
danbart
left a comment
There was a problem hiding this comment.
excelente comienzo revisa y resuelve los comentarios a continuación.
There was a problem hiding this comment.
Hola @jelg14, vamos a intentar mejorar tu código un poco. Listó?
| "credentials": { | ||
| "password": "asdf" |
There was a problem hiding this comment.
| "credentials": { | |
| "password": "asdf" | |
| "credentials": { | |
| "password": "" |
Ingresar la contraseña directamente desde el código es una mala practica de seguridad. Y veo que al parecer tambien la tienes guardado en process.env.PASSWORD. Lo cual es correcto.
Por favor puedes borrar la contraseña del .json. Y realizar los cambios necesarios en tu código para que continué funcionando.
Sugerencia: Para evitar redundancias, mantenerlo modular, y no manipular mucho de los datos dentro del client.ts. Y ya que no se pueden asignar valores desde el datos.json, puedes utilizar un datos.ts para asignar los valores necesarios para el POST. He importar para utilizar en client.ts.
| axios.get(`http://95.217.235.69/${process.env.EMAIL}`,{ | ||
|
|
||
|
|
||
| headers:{ | ||
| "Content-Type": "application/json", | ||
| "x-password": process.env.PASSWORD, |
There was a problem hiding this comment.
| axios.get(`http://95.217.235.69/${process.env.EMAIL}`,{ | |
| headers:{ | |
| "Content-Type": "application/json", | |
| "x-password": process.env.PASSWORD, | |
| axios.get("http://95.217.235.69/"+data.contactInfo.emailAddress,{ | |
| headers:{ | |
| "Content-Type": "application/json", | |
| "x-password": data.credentials.password, |
Para mantenerse consistente, y no causar mal entendidos; como que process.env.EMAIL en GET no es igual a data.contactInfo.emailAddress en POST, y que no estamos buscando dos cosas totalmente diferente.
Sugerencia: podemos utilizar la misma variable importada de datos para llenar los parámetros tanto del POST como el GET.
| "nodemon": "^2.0.4", | ||
| "ts-node-dev": "^1.0.0-pre.44" | ||
| }, | ||
| "prettier": { |
There was a problem hiding this comment.
Veo que ya instalaste prettier pero creo que no se esta aplicando el formato, por los espacios en blanco en tu codigo.
Sugerencia: Puedes colocar el siguiente codigo que me funciono en los setting.json de tu editor:
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
"editor.formatOnSave": true,
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@jelg14 vas por buen camino.
por favor resuelve los comentarios.
@pugaIsaias @danbart asegúrense de darle seguimiento a los cambios de código.
marquen como Resolve Conversation los cambios que se hayan realizado después de sus observaciones.
| export const server = express(); | ||
| server.use(express.json()); | ||
|
|
||
| export default server; No newline at end of file | ||
| export default server; |
There was a problem hiding this comment.
No es necesario modificar este file, por favor devuélvelo a su estado original.
| @@ -5,37 +5,34 @@ | |||
| "title": "The root schema", | |||
There was a problem hiding this comment.
Deja sólo la data necesaria, en este caso tu data.
|
|
||
|
|
||
| async function postData() { | ||
| axios.post('http://95.217.235.69/',datos) |
There was a problem hiding this comment.
| axios.post('http://95.217.235.69/',datos) | |
| axios.post('http://95.217.235.69/', datos) |
ya instalaste prettier?
| @@ -0,0 +1,1875 @@ | |||
| { | |||
There was a problem hiding this comment.
Este file no es necesario. Estamos trabajando con yarn. Quítalo del PR por favor.
4 participants
No description provided.