search

FdelMazo / FIUBA-Plan

Organizador de horarios de la Facultad de Ingenieria
https://fede.dm/FIUBA-Plan/
MIT License
45 stars 6 forks source link

Remove console.debug siuparser #96

Closed lopezac closed 5 months ago

lopezac commented 5 months ago

Hola @FdelMazo , estos console.debug me parece que habria que sacarlos directamente (o quiza comentarlos?) ya que si se quedan en produccion se cargan cada vez que alguien carga el siu, y ralentiza la aplicacion.

FdelMazo commented 5 months ago

Hola Axel, no te seguí del todo. Esos logs influyen en la velocidad de la app? Yo los deje adrede porque creo que si a algún power user no le funciona el parser, va a venir a ver el código al repo, y se va a dar cuenta de que prendiendo las devtools puede tener mas info al respecto.

Como cargar los horarios del SIU es algo que idealmente un usuario solo tiene que hacer una vez por cuatrimestre, no me pareció que le modifique tanto la performance al usuario promedio.

Tenes alguna comparativa de como impactan en la performance?

lopezac commented 5 months ago

Tenes razon, pense que habian sido dejados por accidente, no influye mucho en el rendimiento mirandolo bien.