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

Improve Edt #102

Closed andresmanelli closed 8 years ago

andresmanelli commented 8 years ago

EDT con funcionalidad básica

TODO antes del merge:

Ver README

andresmanelli commented 8 years ago

Update: Agregué lo que faltaba para definir relaciones (como un JTP a una materia). Si largan las query que dejé arriba para ser ADMINS de la Universidad, van a poder probar agregar usuarios asociados. Si se agregan a ustedes mismos, fijense que les aparece ahora eso en "Asociado a:" en el menu de nueva actividad del calendario.

fcladera commented 8 years ago

On Wednesday, March 23, 2016 4:07:21 PM ART, Andrés wrote:

Update: Agregué lo que faltaba para definir relaciones (como un JTP a una materia). Si largan las query que dejé arriba para ser ADMINS de la Universidad, van a poder probar agregar usuarios asociados. Si se agregan a ustedes mismos, fijense que les aparece ahora eso en "Asociado a:" en el menu de nueva actividad del calendario.

Commit flow!

andresmanelli commented 8 years ago

Va con imagen para que venda:

graph

Admito que tal vez haya que cambiar el sistema de que la relación se llame "DECANO" a algo como "MEMBER" con una propiedad que sea el tipo. La que se ve que es de tipo "User", quedó antes de una corrección. No va a pasar de nuevo jaja.

Y el tema de las actividades @francoa dijo que lo simplificaramos, y es buena idea por el tema de la BDD. Creo que a partir de este commit que más o menos termina la base funcional, se pueden definir estas cosas más ordenadamente así avanzamos en robustez y performance.

fcladera commented 8 years ago

La verdad es que soy muy junior en esto de los dibujitos llenos de globitos, pero me gusta

fcladera commented 8 years ago

Che, si podés modificá Travis y el Dockerfile para que carguen el nuevo Dataset.

andresmanelli commented 8 years ago

Pregunto por las dudas: El último punto de la lista de TODO hay que arreglarlo pero técnicamente no forma parte del EDT. Para llegar a la demo no es realmente necesario (aunque sería una funcionalidad más para mostrar). Lo esperan para mergear?

francoa commented 8 years ago

Para mi hagamos merge de esto y dejemos esas funcionalidades para otro branch. Así ya me mando yo tmbn a trabajar en edt sobre una base común.

Aunque si te toma media hora hacerlo, te banco jajaj

andresmanelli commented 8 years ago

Estoy dejando el botón de eliminar, el de editar queda para dsp. Bancame

francoa commented 8 years ago

si, por mientras lo estoy probando

andresmanelli commented 8 years ago

Quedó funcionando el de eliminar, pero hay dos problemas para arreglar: