Jimmygabriel/entry-challenge#16
Conversation
saullopezc
left a comment
saullopezc
left a comment
There was a problem hiding this comment.
Hola @JimmyGabriel
Excelente inicio, resuelve los comentarios a continuación.
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@JimmyGabriel instala prettier en tu editor.
También agrega esta configuración en settings.json.:
"editor.codeActionsOnSave": {
"source.organizeImports": true
},
| }, | ||
| { | ||
| "question": "code is poetry, because:", | ||
| "answer": "The overflow of the soul, a lucid dream, the spirit of the developer. Of what is not yet, but already is, before it was." |
There was a problem hiding this comment.
settings.json , me surge una duda, estoy utilizando webstorm. Recuerdo configurar settings.json en Visual Studio Code, es necesario en WebStorm? Gracias por tus comentarios.
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@saullopezc @areyes107 , @JimmyGabriel hizo actualizaciones, por favor comprueben y hagan nuevos comentarios de ser necesario.
De entrada, los archivos .xml son innecesarios.
Puntos positivos: se aplicó prettier.
hay que checar organización de ./src/data.ts
|
@netpoe , gracias ya atendi tus comentarios sobre la organización de |
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@JimmyGabriel buena iniciativa.
Actualiza tu propuesta con el comentario. Estás cerca.
| switch (process.env.NODE_ENV) { | ||
| case "production": | ||
| path = `${__dirname}/../../.env.production`; | ||
| break; | ||
| default: | ||
| path = `${__dirname}/../../.env.development`; | ||
| } | ||
| dotenv.config({ path: path }); | ||
|
|
||
| export const PASSWORD_ENV = process.env.PASSWORD_ENV; |
There was a problem hiding this comment.
Esta es buena iniciativa, aunque no se recomienda utilizar .env-production y .env-[cualquiera]. Siempre utiliza un solo .env porque en tu sesión sólo hay 1 ambiente.
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
@JimmyGabriel ya casi.
resuelve el último comentario.
Saludos
| @@ -0,0 +1,1197 @@ | |||
| { | |||
There was a problem hiding this comment.
@JimmyGabriel muy cerca
Hay que quitar este file, porque estamos usando yarn para instalar dependencias. yarn crea el archivo yarn.lock, el cual ya existe.
There was a problem hiding this comment.
@netpoe atendi ya tu comentario, gracias. Eliminando package-lock.json.
netpoe
left a comment
netpoe
left a comment
There was a problem hiding this comment.
👍
Bienvenido.
Por favor acepta la invitación de miembro de Github.
También te va a llegar una invitación a Slack
4 participants
POST/GET
Utilizando axio se realizaron los metodos POST y GET dando como resultado claveDeAcceso.