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

Directives are out of app.js! (y regEdtNodes.js se configura en config) #92

Closed andresmanelli closed 8 years ago

andresmanelli commented 8 years ago
francoa commented 8 years ago

@andresmanelli fijate que el archivo config haya quedado como querías porque había conflictos

andresmanelli commented 8 years ago

Antes de mergearla había conflictos? Cómo la mergeaste?

francoa commented 8 years ago

Se resuelven por línea de comandos pero no sé si te dejé los cambios que correspondían (aunque estoy casi seguro que sí)

andresmanelli commented 8 years ago

Bueno, hay que revisar porque no están todos los cambios. Por ahí en estos casos es mejor editar el commit o hacer un rebase (como hizo fer en las anteriores). Hay que revisar el archivo de config. Lo que vi no es de vida o muerte (hay un LOC en esos mensajes de ayuda por ejemplo que habían volado hace unas PR, y un comentario de separación que se había eliminado), pero no sé si hay otras cosas.

francoa commented 8 years ago

El del LOC fue mala mía. El único conflicto estaba entre routes/*.js y routes/user.js. Le habías hecho pocos cambios (me fijé en el historial) y estaba seguro que el merge estaba bien. La única duda estaba con lo de LOC, por eso te pregunté. No me parece que el procedimiento haya sido erróneo. De hecho, así suele hacerse.

andresmanelli commented 8 years ago

Está bien, si eran esos detalles nomás. No sabía lo de línea de comandos, siempre se aprende algo.