PR Client.ts Saul Lopez#10
Conversation
|
@saullopezc Muy buen comienzo amigo, resuelve los comentarios a continuación |
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@saullopezc actualiza tu código para resolver los comentarios. Gracias.
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@areyes107 podrías ayudar a @saullopezc a concretar mejor su propuesta?
Saúl, puedes inspirarte de los demás pull-requests aprobados.
Procura entender el funcionamiento y toma sólo lo necesario.
atención, no sugiero un copy-paste, sino un análisis detenido de los requerimientos.
|
Gracias por el apoyo, se ha cambiado el proceso de extraer de un archivo a variables de entorno |
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
muy cerca.
por favor actualiza con los comentarios recientes.
|
Listo, quedo a la espera de las observaciones Saludos! |
There was a problem hiding this comment.
@saullopezc gracias
por úiltimo, instala https://github.com/prettier/prettier-vscode en tu editor y agrega estas líneas en tus settings.json:
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
una vez instalados, dale save a tus archivos para aplicar el formatting y linting.
|
Los cambios han sido realizados, @netpoe gracias por las observaciones. Saludos |
3 participants
Pull Request para agregar funcionalidad de Cliente utilizando client.js