Skip to content

Incorporar cambios - #33

Open
jacricelli wants to merge 17 commits into
masterfrom
develop
Open

Incorporar cambios#33
jacricelli wants to merge 17 commits into
masterfrom
develop

Conversation

@jacricelli

@jacricelli jacricelli commented Sep 18, 2017

Copy link
Copy Markdown

Referencia: #24, #30, #31, #32

santiagocarello and others added 15 commits August 22, 2017 18:12
'Estudiante invalido' ->

$this->Plataforma->obtenerEncuestado() . ' invalido'
Queda deprecado el ayudante Plataforma en favor de la librería dado
que es necesario algo que pueda utilizarse en toda la aplicación.

Referencia: #31
Los cambios en el commit eaa2985
provocan un error dado que los ayudantes no puede utilizarse en
controladores.
@jacricelli jacricelli self-assigned this Sep 18, 2017
@jacricelli

Copy link
Copy Markdown
Author

@oritoo, @sergioviera, @santiagocarello Estos cambios ya están en el servidor para probarlos en la versión de Tutorías y Graduados.

@oritoo

oritoo commented Sep 19, 2017

Copy link
Copy Markdown
Collaborator

Estuve probando.
Cuando ingresa el administrativo (bvargas) se encuentra con la pantalla principal.
Debe ir a agregar graduado. Lo lleva a la otra pantalla, carga los datos y no lo deja poner el usuario, por lo tanto no aparece la encuesta para cargar; una vez que carga los datos debería de poner el usuario y cuando pone guardar lo lleva a la pantalla de cargar la encuentra.

@sergioviera, @santiagocarello, @jacricelli

@oritoo

oritoo commented Sep 19, 2017

Copy link
Copy Markdown
Collaborator

@sergioviera, @santiagocarello, @jacricelli Por otro lado, necesito que estos cambias sean solo para graduados, se podría?

@sergioviera

sergioviera commented Sep 19, 2017 via email

Copy link
Copy Markdown
Member

@oritoo

oritoo commented Sep 19, 2017

Copy link
Copy Markdown
Collaborator

Bueno, lo hacemos para los dos, total los tutores no tiene los nuevos permisos. me había olvidado. @sergioviera

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants