instrucciones para development

[acrogenesis]

que opinas @HackerOfDreams?

@lalo checa si siguiendo estas instrucciones te funciona :+1:

[ElHacker]

Sobre las versiones que usas. No son 0.9.x son 0.10.x. Estamos usando la version beta de sails. La que tiene Associations.

[acrogenesis]

sudo npm -g install sails@0.10.x npm install sails-disk@0.10.x

asi?

[ElHacker]

si, creo que si

[lalo]

pero porque se tiene que instalar individualmente el sails, que no se baja como dependencia al usar el package.json?

[ElHacker]

@lalo puedes hacer npm install -deso te instala las dependencias del package.json

[juanpabloe]

cuál es el status en este PR? me gustaría instalar el app localmente :dancer:

[acrogenesis]

@juanpabloe intenta con npm install -d para ver si funciona asi

[lalo]

lo unico que le falta (creo) es armar bien el "config/connections.js" de la bd que esta ahorita configurado para heroku y que tu hagas el setup de tu BD

[poguez]

Tienen alguna librería para manejar esas variables de entorno?

[lalo]

no, una de las opciones es modificar el archivo config/connections en la parte de postgres agrengandole algo así:

host: process.env.PG_HOSTNAME || 'localhost',
user: process.env.PG_USER || 'root',
password: process.env.PG_PASSWORD || '***aqui va tu pwd***',
database: process.env.PG_DATABASE || 'name_of_your_db',
port: process.env.PG_PORT || 5432,

y otra es pues exportando las variables antes de correr el app

export PG_USER=eduardo

pero el que le sabe mejor es @HackerOfDreams no se que opine o recomiende esto fue lo que yo hice, ahí mis 2 cents

[lalo]

osea en teoria puedes tener esas variables en tu bash profile y que se carguen siempre entonces eso seria como una "librería" o algo así

[ElHacker]

@lalo WARNING. No hagan ningún commit de keys en config/connections.js. Ese archivo está destinado solo para producción. Lo que se hace en este caso es modificar el archivo config/local.jspongan esto al final.

connections: {
    postgresql: {
      adapter   : 'sails-postgresql',
      host      : 'localhost',
      user      : '**aqui va tu user**',
      password  : '**aqui va tu password**',
      database  : 'cambia_la_ley'
    }
  }

Este archivo está en el .gitignore por lo tanto cada uno de ustedes tiene que modificar el archivo config/local.js

[lalo]

:+1:

[poguez]

:+1:

[ElHacker]

@acrogenesis ¿Te animas a poner esto sobre el branch repo-guidelines?, está referenciado en el issue #28

[ElHacker]

@acrogenesis Estoy viendo que hiciste los cambios en el README.md y lo pusiste en inglés. Hechale un ojo al issue #28. El archivo README.md debe de estar en Inglés y el LEEME.md en español. ¿Tienes chance de ponerlo porfa?, esta sobre el branch repo-guidelines

[acrogenesis]

Abri un nuevo pull request para hacerlo a otra branch #32