Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generar horario compartido en el servidor #43

Open
benjavicente opened this issue Jul 29, 2021 · 6 comments
Open

Generar horario compartido en el servidor #43

benjavicente opened this issue Jul 29, 2021 · 6 comments
Labels
feature New feature or request

Comments

@benjavicente
Copy link
Member

Hay un problema en el funcionamiento en el sistema de compartir horarios: estos se obtienen por parte del cliente con 1 request por cada entrada del horario, lo que hace imposible crear tags meta correspondientes y también añade complejidad innesesaria.

$(() => {
var h = "{{ ids }}";
h = h ? h.split(',') : [];
h.forEach((id) => {
wp.loadRamo(id, false);
})
})

Y el controlador para compartir, share, está en el mismo archivo de schedule, el que se usa para obtener el horario!

Hay que integrar la lógica de obtención de los horarios en share para generar la página el horario compartido en el servidor. Talvez crear un filtro que tome una lista de secciones y genere la tabla correspondiente sea buena idea.

@nico-mac
Copy link
Member

share usa schedule justamente porque se comparten horarios. Se podría dejar similar a como esta pero haciendo la un solo request para múltiples secciones a la vez.

También se puede aprovechar de cambiar el uso de ids en el link por siglas. esto incluso se puede hacer manteniendo compatibilidad con versiones anteriores si se usan parámetros distintos a h.

@benjavicente
Copy link
Member Author

benjavicente commented Jul 30, 2021

Se podría dejar similar a como esta pero haciendo la un solo request para múltiples secciones a la vez.

Pero al compartir los horarios no debería ser necesario agregarlos: es como si encargaras una pizza 🍕, pero al momento de recibirla te pasan solo la masa y tienes que pedir X nuevos pedidos (cada request adicional) para agregar cada uno de los X ingredientes de la pizza. La pizza debería estar lista al salir de la pizzería!

También se puede aprovechar de cambiar el uso de ids en el link por siglas. esto incluso se puede hacer manteniendo compatibilidad con versiones anteriores si se usan parámetros distintos a h.

Eso no sería más largo o complejo? Porque tengo entendido que el id del curso muestra un curso que se hace en un periodo académico en específico, utilizar las siglas (MAt1640-1 por ejemplo) requeriría añadir el periodo en el que se realizó el curso y luego buscar los cursos segun su sigla y periodo.

Edit: Me acordé que eso hago así en UCalendar. Podría resultar en links más bonitos, pero tal vez muy largos.

@nico-mac
Copy link
Member

No es que sea necesario agregarlos, es que se hace un request por la información del horario de cada id (tipo API JSON), en lugar de usar templating. Lo óptimo sería en un futuro evitar el templating en favor de una API y un framework de frontend.

¿Que condición se necesita para hacer meta tags?

@benjavicente
Copy link
Member Author

Que el que entrega el html tenga la información, por ejemplo, en el caso de los cursos, que sepa curso es el que se mostrará en la página.

@nico-mac
Copy link
Member

¿Tiene que ser con templating entonces?

@benjavicente
Copy link
Member Author

Sip, creo que no hay otra forma

@benjavicente benjavicente added the feature New feature or request label Aug 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants