Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #58 port already used #97

Merged
merged 8 commits into from
Oct 4, 2023
Merged

Fix #58 port already used #97

merged 8 commits into from
Oct 4, 2023

Conversation

fdodino
Copy link
Contributor

@fdodino fdodino commented Oct 3, 2023

En este PR metemos algunas cositas:

  • escuchamos el evento 'error' para que si el REPL no puede levantar el diagrama dinámico en el 3000 te tire un mensaje copado
  • de paso, si hay otro tipo de error también mostramos un mensaje
  • la alternativa es que le pases un puerto distinto: --port xxxx y eso está aclarado en el mensaje de error "copado"
  • la página del diagrama hardcodeaba el puerto 3000 con lo cual daba un error de CORS al conectarse con el server (igual lo que iba a hacer era espejar el diagrama del 3000 en otras páginas). Ahora podés tener n REPLs con sus correspondientes diagramas dinámicos, una locura (no se para qué pero se puede)
  • y también deshabilitamos mensajes que aparecían cada dos por tres en el REPL diciendo que se conectaba/desconectaba el diagrama dinámico. Esto es esperable vía socket, porque timeoutea y después se reconecta solito.

Va una demo para @JuanFdS @asanzo y la muchachada

dynamicDiagramDuo

De paso le metí una mejora a la explicación del toggle que hace que los objetos se reubiquen o no.

@fdodino fdodino added the componente: consola Interpreta los comandos de la consola label Oct 4, 2023
Copy link
Contributor

@PalumboN PalumboN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GÑial!!! 🤩🎉

A mí me interesaba el tener N clientes viendo la misma consola. Me lo imagino formato labo dónde todos ven el diagrama en sus compus.

Lo de tener N servers también está piolita, no sé para qué jajaj

@fdodino
Copy link
Contributor Author

fdodino commented Oct 4, 2023

GÑial!!! 🤩🎉

A mí me interesaba el tener N clientes viendo la misma consola. Me lo imagino formato labo dónde todos ven el diagrama en sus compus.

Está piola pero habría que toquetear la página para que no hardcodee localhost sino window.domain. Y de paso estaría bueno poder conectarse como cliente a un REPL existente.

@fdodino fdodino merged commit 7d58786 into master Oct 4, 2023
1 check passed
@fdodino fdodino deleted the fix-#58-port-already-used branch October 4, 2023 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
componente: consola Interpreta los comandos de la consola
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants