Jimmygabriel/entry-challenge #16
Conversation
There was a problem hiding this comment.
Hola @JimmyGabriel
Excelente inicio, resuelve los comentarios a continuación.
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
},
src/data.ts
Outdated
| }, | ||
| { | ||
| "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.
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 |
There was a problem hiding this comment.
@JimmyGabriel buena iniciativa.
Actualiza tu propuesta con el comentario. Estás cerca.
src/utils/config.ts
Outdated
| 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.
There was a problem hiding this comment.
@netpoe gracias, atendi ya tu comentario.
There was a problem hiding this comment.
@JimmyGabriel ya casi.
resuelve el último comentario.
Saludos
package-lock.json
Outdated
| @@ -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.
POST/GET
Utilizando axio se realizaron los metodos POST y GET dando como resultado claveDeAcceso.