Cambios UI: siempre pedir legajo y confirmar si la entrega es grupal #54
Conversation
| @@ -90,7 +108,7 @@ | |||
| <script> | |||
| document.addEventListener("DOMContentLoaded", function() { | |||
| $('#tp').on('input', validate); | |||
| $('#identificador').on('input', validate); | |||
| $('#legajo').on('input', validate); | |||
There was a problem hiding this comment.
Ojo, en el label hay que poner padrón, porque si bien son lo mismo en la facu usamos esa palabra
There was a problem hiding this comment.
En la label hice esto:
algo2_sistema_entregas/templates/index.html
Lines 18 to 21 in d069e0f
O sea sostuve Legajo como etiqueta, pero puse Legajo o padrón como placeholder (o sea, el texto visible en la text box cuando aún no se ha escrito nada).
Si no te parece suficiente, lo cambio; pero yo mencionaría ambos términos, porque me dio la impresión que algunes alumnes de generaciones recientes no oyeron nunca padrón, y sí legajo.
Personalmente, mencionando ambos en el placeholder me parece suficiente, pero si no estás de acuerdo, lo cambio.
There was a problem hiding this comment.
Ah, perfecto, no había visto eso. Dejemoslo así, y si surjen preguntas vemos
Con vistas a definir con precisión a qué repositorio se debe importar el código (ver algoritmos-rw#49), en este commit se comienza por: - siempre pedir el legajo a la persona que está subiendo el código - para entregas grupales, hacer explícita la elección de si el código se sube a título individual, o grupal (antes esto iba implícito según se usase padrón o número de grupo como identificador) Cambios en la UI: - se renombra el campo Identificador, a Legajo (el número de grupo se muestra, si lo hay, pero nunca se pregunta explícitamente) - se introduce un selector "radio" que, para trabajos grupales, permite cambiar la modalidad a entrega a título personal. Refactorings internos: - el JSON de correctores que se envía a index.html pasa a tener como claves solamente legajos (eliminando claves GXX y gYYYY); los valores son ahora un arreglo que especifica correctores (individual y grupal), y número de grupo. Esto facilita poder seguir mostrando los nombres de corrector, y además saber en O(1) el número de grupo asociado a un legajo.
El selector entre individual/grupal se desactiva automáticamente para entregas individuales por una cuestión de UX. Pero eso, por la semántica de HTML, hace que el valor no se envíe. Por tanto, se debe detectar en el servidor su ausencia, y que solo se produce en el caso permitido (entregas individuales). Se puede intentar forzar a que se envíe el elemento, pero la validación en el backend lo convierte en superfluo igualmente.
Con vistas a definir con precisión a qué repositorio se debe importar
el código (ver #49), en este commit se comienza por:
siempre pedir el legajo a la persona que está subiendo el código
para entregas grupales, hacer explícita la elección de si el código
se sube a título individual, o grupal (antes esto iba implícito según
se usase padrón o número de grupo como identificador)
Cambios en la UI:
se renombra el campo Identificador, a Legajo (el número de grupo
se muestra, si lo hay, pero nunca se pregunta explícitamente)
se introduce un selector "radio" que, para trabajos grupales,
permite cambiar la modalidad a entrega a título personal.
Refactorings internos:
el JSON de correctores que se envía a index.html pasa a tener
como claves solamente legajos (eliminando claves GXX y gYYYY);
los valores son ahora un arreglo que especifica correctores
(individual y grupal), y número de grupo.
Esto facilita poder seguir mostrando los nombres de corrector,
y además saber en O(1) el número de grupo asociado a un legajo.
This change is