Tus salas ya no se caen del menu por el tope del navegador - #77
Merged
Merged
Conversation
Una sala de hace dos dias desaparecio del menu aunque seguia viva. No fue que faltara guardarla: el server ya la tenia, y la sincronizacion al arrancar ya bajaba la lista completa de la cuenta. El problema era que `guardarSalas` la recortaba a veinte antes de escribirla, asi que la cuenta servia solo para las primeras veinte y en la siguiente sincronizacion se subian nada mas esas. Las demas seguian en la base sin que nadie las volviera a ver. Tres cambios, todos en el cliente: - `guardarSalas` deja de recortar. Ese tope es para una lista que crece sola visita tras visita, no para una que el server ya tiene completa. - El tope local sube de veinte a cien. Veinte se desbordaba en unos dias de trabajo normal, y lo que se caia no era la sala que menos usabas sino la mas vieja, que puede ser justo la que buscabas. - El menu pregunta al server al abrirse, no solo al arrancar la pagina. Primero pinta lo local, que es instantaneo, y lo reemplaza cuando llega la respuesta. Sin sesion devuelve lo mismo que ya se veia, asi que no cambia nada. El DELETE de `auth/salas` sigue sin usarse y esta bien: hoy solo se quita del menu una sala que se borro de verdad, y esa fila se cae sola por la llave foranea. Hara falta el dia que se pueda sacar una sala de tu lista sin destruirla. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01L1nFf8JaKeGedJ6RaZQYZ8
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Una sala de hace dos dias desaparecio del menu aunque seguia viva. No fue que faltara guardarla: el server ya la tenia, y la sincronizacion al arrancar ya bajaba la lista completa de la cuenta. El problema era que
guardarSalasla recortaba a veinte antes de escribirla, asi que la cuenta servia solo para las primeras veinte y en la siguiente sincronizacion se subian nada mas esas. Las demas seguian en la base sin que nadie las volviera a ver.Tres cambios, todos en el cliente:
guardarSalasdeja de recortar. Ese tope es para una lista que crece sola visita tras visita, no para una que el server ya tiene completa.El DELETE de
auth/salassigue sin usarse y esta bien: hoy solo se quita del menu una sala que se borro de verdad, y esa fila se cae sola por la llave foranea. Hara falta el dia que se pueda sacar una sala de tu lista sin destruirla.Claude-Session: https://claude.ai/code/session_01L1nFf8JaKeGedJ6RaZQYZ8