Edit profile #60
Edit profile #60
Conversation
| @@ -128,7 +129,7 @@ | |||
| 'USER': os.getenv('POSTGRES_USERNAME', 'root'), | |||
| 'PASSWORD': os.getenv('POSTGRES_PASSWORD', 'example'), | |||
| 'HOST': os.getenv('POSTGRES_HOST', '127.0.0.1'), | |||
| 'PORT': os.getenv('POSTGRES_PORT', 5432), | |||
| 'PORT': os.getenv('POSTGRES_PORT', 5433), | |||
There was a problem hiding this comment.
Ojo con este tipo de pushs! Dejar el puerto por default o le va a explotar a todos
There was a problem hiding this comment.
si fue mala mia. Lo arreglé en el otro commit que pushee con este, se hizo?
There was a problem hiding this comment.
Nop, no aparece! En el commit que hiciste corregiste el docker-compose.dev.yml, pero no este!
There was a problem hiding this comment.
ahora si, disculpame. No me aparecia el cambio en ese archivo.
| @@ -151,6 +152,8 @@ | |||
| }, | |||
| ] | |||
|
|
|||
| MIN_PASSWORD_LEN: int = 8 | |||
There was a problem hiding this comment.
No dejar hardcodeado! Sacar de las variables de entorno (las variables de entorno son aquellas que se setean desde el docker-compose.yml y nos permite parametrizar el sistema sin cambiar el código). Hacer como se hace más abajo en este archivo con la variable N_JOBS_RF. Fijate que la obtenemos con os.getenv, le ponemos un valor por defecto y lo casteamos a entero, hacer lo mismo acá
src/users/models.py
Outdated
| @@ -0,0 +1,3 @@ | |||
| from django.db import models | |||
There was a problem hiding this comment.
No dejar ninguna línea en este archivo, así no se importa models al pedo ralentizando el inicio del sistema
No description provided.