search

lugfi / organizador-fiuba

El organizador te ayuda a organizar las materias con un calendario semanal, mostrando superposiciones y permitiendo probar distintas combinaciones de cursos rapidamente.
https://lugfi.github.io/organizador-fiuba/
GNU General Public License v3.0
15 stars 16 forks source link

Subo el script php de Franco #32

Closed edittler closed 7 years ago

edittler commented 7 years ago

Cree un directorio "tools" para meter estas utilidades que no influyen en la funcionalidad misma de la app.

Más aún, considero que el parser.js debería estar en ese directorio. Si están de acuerdo muevo el archivo antes de que aprueben el PR.

Resolves #27

colltoaction commented 7 years ago

Please movelo! Eso sí, habría que actualizarlo.

Por otro lado, no vi si Franco me respondió la pregunta sobre cómo leía el pdf. En este script solo veo lectura de un archivo de texto. Quizás copiaba y pegaba, pero ni idea.

edittler commented 7 years ago

Ya moví el parser.js y cambié los paths. No pude probarlo, a la noche podría hacerlo si es que nadie lo hace antes.

Sobre cómo funciona el script de PHP podría comentarnos @lucasp90 que ayer lo usó.

AlexDaciuk commented 7 years ago

Me parece bien, mergeo, si hay algun tema con el script de php, se abre un issue aparte

AlexDaciuk commented 7 years ago

Tambien saque los datos personales de Franco del script

colltoaction commented 7 years ago

Crack, me había olvidado de eso.

On Thu, Aug 3, 2017, 11:06 AM Alexis Daciuk notifications@github.com wrote:

Merged #32 https://github.com/lugfi/organizador-fiuba/pull/32.

— You are receiving this because your review was requested.

Reply to this email directly, view it on GitHub https://github.com/lugfi/organizador-fiuba/pull/32#event-1191895484, or mute the thread https://github.com/notifications/unsubscribe-auth/ABeg9G8bGG3RklP2qQkqfXrT2zElS_AKks5sUgwcgaJpZM4Or6sh .