Feat/borrar sala - #28
Merged
Merged
Conversation
Al componente de la Sala le faltaba `key`, asi que al cambiar de sala React
veia el mismo componente en el mismo sitio, reusaba la instancia y conservaba
su estado: los mensajes, los agentes y el preview de la sala ANTERIOR.
En una sala con historial no se notaba, porque el `joined` llegaba con mensajes
y pisaba lo viejo. En una recien creada si: llega vacio, nada sobrescribe, y
acababas en tu sala nueva leyendo la conversacion de otra.
Con `key={roomId}` React remonta al cambiar de sala. El socket viejo se cierra
solo, porque el efecto ya hacia `disconnect` al desmontar.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
La X del menu quitaba la sala del localStorage y ya: el proyecto, el chat y el contenedor seguian ahi para siempre. Si la quitas de la lista es porque ya no la vas a usar, asi que ahora se borra completa. Se va todo: el workspace en disco con sus hermanos (.adjuntos, .turns.json, .bookmarks.json), el contenedor, y las tres tablas de la BD. El orden importa: primero se apaga lo que corre, porque borrar el directorio con el dev server vivo y el contenedor montado deja procesos escribiendo en archivos que ya no existen. Se pregunta antes, con el nombre de la sala en la pregunta: es irreversible y NO es personal. La sala es de quien tenga el link, asi que borrarla se lleva el trabajo de los compas que esten ahi. A quien este dentro se le avisa por socket (`room:deleted`), porque si no se quedaba con un preview que ya no responde sin saber por que. Si el server falla, la sala NO se quita de la lista: seguiria existiendo, y esconderla del menu solo taparia algo que sigue ocupando disco. Probado en local: se borro una sala con adjuntos y no quedo rastro ni en disco, ni en contenedores, ni en la BD (el server responde "sala no encontrada"). Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
2 participants
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.
No description provided.