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

Using web tools component #191

Merged
merged 15 commits into from
Oct 9, 2024
Merged

Using web tools component #191

merged 15 commits into from
Oct 9, 2024

Conversation

fdodino
Copy link
Contributor

@fdodino fdodino commented Sep 28, 2024

Hay que releasear wollok-web-tools, pero algunas cosas que metí en este PR

  • subo la versión de Node a lts/iron, la 20 LTS (raro que compilamos con la 18 y les pedimos a la gente que instalen la 20)
  • subí la versión del eslint a la 8, que trae un cambio en el formato (quotes, semicolon => lo mandaron a otro plugin styllistic)
  • en el npm install copio las referencias para game y dynamic diagram, los nombres los estandaricé a game-index y diagram-index, se copian a public para poder ser usados
  • en el build no hay que hacer nada, ya están los archivos en /public y se copian a /dist/public
  • como en wollok-web-tools metí un entry point que sea index.ts (los otros tienen un nombre distinto), eso me permite poner las dependencias de módulos para poder ser importado directamente:
import { VALID_IMAGE_EXTENSIONS, VALID_SOUND_EXTENSIONS } from 'wollok-web-tools'
import { getDataDiagram } from 'wollok-web-tools'

no hay que hacer import { getDataDiagram } from 'wollok-web-tools/dist/dynamicDiagram/diagram-generator'

  • generé un util para evitar el problema que se generaba si pedía el diagrama dinámico desde wollok-web-tools, donde por algún motivo que no pude entender en wollok-ts el intérprete le pregunta a las referencias de la consola si son Variable y dice que no. Eso causa que las referencias del REPL no aparezcan (solo aparecen los objetos):

image

En el util llamo desde ts-cli directamente a wollok-ts y obtengo los objetos para después llamar a wollok-web-tools con esos objetos.

  • Los tests pasaron por arte de magia con solo cambiar un par de imports
  • El archivo diagram-index.js asume que existe cytoscape importado dentro del html, se copia en el npm install y listo
  • También renombré las dependencias a p5 y p5sound como minificadas, pensé en un momento que faltaba hacer eso pero solamente tenían el nombre normal.
  • Eliminamos el archivo diagram-generator.ts que adapta la información como viene de wollok-ts a lo que cytoscape necesita. También se elimina el archivo diagram.js que se renombra a diagram-index.js. Ambos archivos ahora están bajo la órbita de wollok-web-tools para poder reutilizarse en el sitio web.

@fdodino fdodino marked this pull request as draft September 28, 2024 18:21
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.

Tale tale tale 🪵 🪓

package.json Outdated
@@ -61,9 +62,10 @@
"socket.io": "^4.5.1",
"winston": "^3.11.0",
"wollok-ts": "4.1.6",
"wollok-web-tools": "1.1.2"
"wollok-web-tools": "file:../wollok-web-tools"
Copy link
Contributor

Choose a reason for hiding this comment

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

Actualizar después de releasear tools

@fdodino fdodino marked this pull request as ready for review October 9, 2024 00:15
@fdodino fdodino merged commit c9f197f into master Oct 9, 2024
1 check passed
@fdodino fdodino deleted the using-web-tools-component branch October 9, 2024 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants