FdelMazo / FIUBA-Plan

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

Exportar (y porque no, importar) Planes #59

Closed Fxusto closed 1 year ago

Fxusto commented 2 years ago

Estaria genial poder exportar los planes que armas, podrian exportarse como una hoja de excel o en su defecto en json. Pero para poder almacenarlos de alguna forma.

Dejo esto por si es de algun tipo de ayuda. https://www.npmjs.com/package/react-export-excel

FdelMazo commented 2 years ago

Hola @Fxusto !

Gracias por el issue. Me ayudas a entenderlo mejor? Cual sería un contexto en el que necesite exportarlo? decis para luego poder volver a cargarlos en el Fiuba Plan, o para tenerlo como json/excel y llevartelo a otro lado?

En este momento el plan se guarda en tu browser session, entonces cada vez que entres desde la misma PC vas a ver el mismo plan. No se me ocurre bien para que pasarlo a otra plataforma! En un json me parece que es bastante ilegible, y si es cuestion de tenerlo en el celu, creo que podria tranquilamente pasarse por screenshot.

Tal vez guardar todo el plan en un hash, cosa de que puedas copiar y pastear esa cadena en el fiuba map y se te re-hace el plan?

Fxusto commented 2 years ago

La idea de hashearlo esta buena me parece hasta mas practico, mas que nada lo pensaba para poder compartirlo sin tener que sacar screenshot y perder el dinamismo de poder seguir modificando cosas. Ya sea para compartirlo con otra persona o poder tenerlo en otra PC sin tener que volver a armar todos los planes desde cero.

FdelMazo commented 2 years ago

Genial che! Dejo el issue abierto y pienso alguna forma intuitiva de meterlo. No creo poder darle bolilla pronto. Si te animas, mandate y codealo! Los PRs son siempre bienvenidos

FdelMazo commented 2 years ago

Realmente no la veo sencilla la de compartir un plan así como así. En particular conflictua bastante con lo que el usuario ya tiene en su cache del browser.

Digo, imaginemos un usuario que tiene su plan de informatica en su browser. Un amigo le manda su plan de industrial para mostrarle lo que va a cursar. Que haces ahi? Juntas ambos, pisas uno, etc... Es dificil!

Encima, te cae el plan de industrial, y de la nada tenes que re-armar el resto de la app para que tenga sentido! El plan de industrial van a ser N cursos, pero en el sidebar de la derecha tenes que poner ahora las M materias para que puedas editarlas, y eso va a ser un bardo!

Entonces pienso... y si se comparte en read-only un plan?

y ahora la pregunta del millon. Realmente tiene valor esto? es exactamente la misma funcionalidad que sacarle un screenshot a tu plan y guardartelo en el celular!

Que opinas @Fxusto ?

A lo que voy es... que estamos queriendo solucionar?

julitaras commented 2 years ago

Yo creo que esta bien si es una especie de readonly, ya que es el plan de uno y el otro quizas omite alguna opinion. O en caso de que se quiera con mas permisos uno decida (esto probablemente sea ir muy lejos).

Ejemplo: Queria compartirle a un amigo los planes que tenia en caso de aprobar el final de una materia, pero eran muchos planes.... por lo que el screen se me volvio una verdadera paja, ya que eran varias opciones. Entonces si hubiese algun tipo de share estaria piola. Quizaas estaria piola lo que mencionas que tiene el fiuba map para que ademas este syncronizado con el resto de los dispositivos

julitaras commented 2 years ago

Me conformo con el readonly 😆

FdelMazo commented 1 year ago

Bueno... termine haciendo algo medio raro con el local storage y las URL compartibles. Agregué este botón al sitio, en ed885da811ced7b55e48617382c44c9bc5af8715: image

@julitaras @Fxusto Tienen ganas de probar este exageradamente beta feature? Es clickear el boton, y compartirle la URL a algún otro dispositivo/amigo y rezar que ande todo

Fxusto commented 1 year ago

Estos días lo estuve probando, tanto para pasarme planes entre PC y PC como con un par de compañeros para compartirnos los planes y funciona bien. A veces cuando ya estas en fede.dm/FIUBA-Plan/ y pegas el permalink nuevo hace falta recargar la pagina para que lo tome, pero supongo que eso tiene que ver con algo del browser. Me gusta que este el alert con el permalink actual para no perderlo pero al principio me pareció que quedaba un poco agresivo por el link con el hash tan largo. En general me gusta como lo resolviste y me parece una feature muy útil. Yo lo probé siempre desde Chrome.

FdelMazo commented 1 year ago

Gracias @Fxusto. Cierro el issue, si se te ocurren cosas para meterle no dudes en abrir otro. Gracias por la idea y por probarlo!

julitaras commented 1 year ago

Buenisimo Fede! Esta muy bueno el warning para que no se pisen los planes