MecatronicaUncu / Red-Social-Asociacion

A small open source social network for any small community
GNU General Public License v2.0
3 stars 1 forks source link

README.md improvements - Fixes #99 #107

Closed fcladera closed 8 years ago

fcladera commented 8 years ago
andresmanelli commented 8 years ago

Muy bueno.

Podrás cambiar esto?:

Para Develop tal vez poner grunt watch ?

fcladera commented 8 years ago

Yo creo que ahí estaría, revisalo por las dudas!

andresmanelli commented 8 years ago

Te puedo molestar con algo más? Borrá mejor las líneas 158 a 162 inclusive, porque users.js y _users.js son casos particulares de moduleName.js y _moduleName.js

Y si no vas a cambiar nada con lo que ibas a probar de docker, lo mergeo a menos que @francoa quiera agregar algo más

francoa commented 8 years ago

Son muy diferentes las instrucciones para correr en desarrollo que en producción? Si afirmativo, sería útil que quede explicado (pensando en futuras incorporaciones)

andresmanelli commented 8 years ago

En teoria en production deberia ser grunt compile y node server.js (creo que mas arriba lo puse) pero nunca lo probamos. Franco podes ver si eso funciona?

-----Message d'origine----- De : "Franco Ardiani" notifications@github.com Envoyé : ‎30/‎03/‎2016 19:34 À : "MecatronicaUncu/Red-Social-Asociacion" Red-Social-Asociacion@noreply.github.com Cc : "Andrés" andresmanelli@gmail.com Objet : Re: [MecatronicaUncu/Red-Social-Asociacion] README.md improvements -Fixes #99 (#107)

Son muy diferentes las instrucciones para correr en desarrollo que en producción? Si afirmativo, sería útil que quede explicado (pensando en futuras incorporaciones) — You are receiving this because you commented. Reply to this email directly or view it on GitHub

fcladera commented 8 years ago

Todavía no tenemos producción... no lo hemos probado! En realidad cuando tengamos un docker de producción va a ser automático todo.

fcladera commented 8 years ago

Pero yo no perdería tiempo en esto haha. Si quieren abran un issue de "probar en producción, crear dockerfile de producción" y si lo necesitamos alguna vez lo arreglamos

fcladera commented 8 years ago

En teoría, cuando estemos en producción deberíamos cambiar la dirección de la img Docker en la sección "Run in Linux", para que sea algo así como

-docker run -p 3000:3000 -t -i mecatronicauncu/red-social-asociacion-dev 
+docker run -p 3000:3000 -t -i mecatronicauncu/red-social-asociacion-prod

pero como no tenemos todavía ningún red-social-asociacion-prod, lo haremos cuando eso exista. Insisto que no hay ningún apuro.

francoa commented 8 years ago

joya, entonces está todo indicado para dev. No comments

fcladera commented 8 years ago

Ya pusheo los cambios que pidió RM

fcladera commented 8 years ago

Done