68 editar datos del alumno

[NahuelM426]

Closes #68
hay cosas comentadas en el controller y CargarInscripcionPage que de EstudiosCursados

[vgrana]

Bajé la rama, y veo que te falta el inscribir cursada a un alumno.

[vgrana]

Teléfono, a la hora de editarlo, trae el campo vacio;
Me parece que la carrera no debería editarse, sino mostrarse

[vgrana]

Recordar que hay código comentado
Ver definición de si la carrera en la que está inscripto el alumno debería poder editarse

[faloi]

Me parece bien que la carrera pueda editarse, aunque deberíamos agregar una validación que no permita hacerlo si ya está cursando alguna materia. Lo pongo en una issue: #160.

Como dijo @vgrana, borrá el código comentado y mergeamos.

[NahuelM426]

Bueno Fede El lun., 15 de abr. de 2019 10:51, Federico Aloi <[email protected]> escribió:

…

***@***.**** requested changes on this pull request. Me parece bien que la carrera pueda editarse, aunque deberíamos agregar una validación que no permita hacerlo si ya está cursando alguna materia. Lo pongo en una issue: #160 <#160>. Como dijo @vgrana <https://github.com/vgrana>, borrá el código comentado y mergeamos. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#159 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AKLROJNPUFRFOBJ67XSVYZTPQSB5FANCNFSM4HF3K4SQ> .

[faloi]

Resolvé los conflictos y mergeá, ya está aprobado.