cokkike88 pull request#11
Conversation
Ktoxcon
left a comment
Ktoxcon
left a comment
There was a problem hiding this comment.
Muy Buen Comienzo @cokkike88 !! 🎉🎉🎉
Por favor resuelve lo que se te pide en los comentarios.
| }); | ||
|
|
||
| server.get("/:emailAddress", async (req: Request, res: Response) => { | ||
| server.get("/:emailAddress", async (req, res) => { |
There was a problem hiding this comment.
No hay necesidad de modificar -/src/index.ts. Por favor, regresa este file al estado original.
Puedes intentar hacer git checkout master src/index.ts para regresar este file a como estaba antes. 😉
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
Gracias @Ktoxcon por los comments.
@cokkike88 resuelve los comentarios, por favor.
| server.listen("80", () => { | ||
| server.listen("8081", () => { | ||
| console.log("listening"); | ||
| }); |
There was a problem hiding this comment.
No es necesario modificar index.ts, por favor devuélvelo al estado original.
| @@ -0,0 +1,1178 @@ | |||
| { | |||
There was a problem hiding this comment.
package-lock.json es creado por npm, pero en este proyecto estamos usando yarn. Cuando instales dependencias utiliza yarn.lock por favor. Y remueve este file.
|
|
||
| router.post('/sendInformation', async(req, res) => { | ||
| try { | ||
| let data = { |
There was a problem hiding this comment.
Mueve let data a un file ./src/data.ts para simplificar client.ts.
Igualmente, hay que usar const data, por que no se reasignará en el futuro.
| username: "cokkike88", | ||
| }, | ||
| credentials: { | ||
| password: "Enrique8@Lisso12", |
There was a problem hiding this comment.
Evita subir contraseñas a repositorios públicos (y privados), es una mala práctica. Cómo podemos hacer para ocultar esta contraseña?
There was a problem hiding this comment.
Se instalo la libreria dotenv, para configurar varaibles de entorno y se ignora el archivo de configuracion en el .gitignore. Ya subi los cambios.
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@cokkike88 todavía veo algunos de los comentarios no resueltos, como las observaciones sobre el file index.ts.
Por favor instala: https://github.com/prettier/prettier-vscode en tu editor.
Igualmente, agrega estas líneas en tu settings.json:
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
| questions: [ | ||
| { | ||
| question: "If I was a Sr. Programmer, I would like to build:", | ||
| answer: "Respuesta: Some application with micriservices with aws lambda, with neptuno database, nodejs as backend, reactjs as frontend, aws cognito for authentication and authorization, aws s3 to resources storage, aws api gateway", |
There was a problem hiding this comment.
hay que poner atención a los detalles
| answer: "Respuesta: Some application with micriservices with aws lambda, with neptuno database, nodejs as backend, reactjs as frontend, aws cognito for authentication and authorization, aws s3 to resources storage, aws api gateway", | |
| answer: "Respuesta: Some application with microservices with aws lambda, with neptuno database, nodejs as backend, reactjs as frontend, aws cognito for authentication and authorization, aws s3 to resources storage, aws api gateway", |
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@cokkike88 te falta agregar el código del cliente que devuelve la claveDeAcceso.
| server.post('/sendInformation', async(req, res) => { | ||
| try { | ||
| let result = await axios.post(url, data); | ||
| res.json(result.data); | ||
| } | ||
| catch(error){ | ||
| return error; | ||
| } | ||
| }); | ||
|
|
||
|
|
||
| server.listen("8081", () => { | ||
| console.log('listening 8081'); | ||
| }) No newline at end of file |
There was a problem hiding this comment.
En tu solución no veo el GET request que se tiene que hacer para recibir la claveDeAcceso. 🤔
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@cokkike88
Muy cerca.
Por favor instala prettier en tu editor y guarda los files de nuevo para aplicar el nuevo formato.
También agrega esta configuración a tu settings.json:
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
…ivos, y se agrego la configuracion en settings.json de vscode
| server.post('/sendInformation', async (req, res) => { | ||
| try { | ||
| let result = await axios.post(url, data); | ||
| res.json(result.data); | ||
| } | ||
| catch (error) { | ||
| return error; | ||
| } | ||
| }); |
There was a problem hiding this comment.
@cokkike88 todavía veo una sangría incorrecta. Ya instalaste prettier en tu editor?
hay que guardar los files de nuevo para que se aplique el formatting.
4 participants
:)