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

Reemplazar RequireJS por Vite/Rollup/Webpack #29

Closed
ominousg opened this issue Jul 28, 2022 · 1 comment
Closed

Reemplazar RequireJS por Vite/Rollup/Webpack #29

ominousg opened this issue Jul 28, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@ominousg
Copy link
Owner

AOWeb usa RequireJS, un cargador de módulos/package manager que usa el patrón de importación AMD, y no es compatible con las versiones nuevas (+v6) de Pixi JS. Todo lo que se mostró en el video -deferred shading, partículas, shaders, texturas comprimidas con KTX, animación esquelética, etc- necesita pasar de la versión 4.0.3 de Pixi a +v6 para funcionar correctamente, por lo tanto este issue (migrar de RequireJS) tiene la prioridad más alta.

Dejo un poco de información por si alguien quiere intentar la migración a Vite/Rollup o Webpack - también me va a servir para refrescar la memoria en un futuro, si es que todavía no se migró.

Ejemplos/boilerplates usando Pixi JS: Vite + Pixi v6 - Rollup + Pixi v6 - Webpack + Pixi v5 - Otro Webpack + Pixi v5

RequireJS se encuentra en js > lib > require.js - El AOWeb carga los modulos en js > home.js y está declarado en el index.html de la siguiente manera:

<script data-main="js/home" src="js/lib/require.js"></script>

Otra parte es actualizar todas las importaciones en todos los archivos que usan define estilo AMD - esto ya lo tengo hecho, si alguien va a intentar la migración me lo puede pedir por acá. Por ejemplo en el caso de js > model > acciones.js pasar de:

define(["enums", "font", "model/macros"], function (Enums, Font, Macros) {

a:

import * as Enums from "../enums";
import * as Font from "../font";
import * as Macros from "./macros";
@ominousg ominousg added enhancement New feature or request help wanted Extra attention is needed labels Jul 28, 2022
@ominousg ominousg pinned this issue Jul 28, 2022
@ominousg
Copy link
Owner Author

ominousg commented Feb 4, 2023

🥳 Terminado: 158eedf. Migración de RequireJS a Webpack. Ahora se pueden usar e instalar dependencias con npm. Se cambió el estilo de importación AMD (define) por ESM (import). Trasladando HTML de los popups a sus archivos .js

Probablemente haga una migración a Vite en el futuro.

@ominousg ominousg closed this as completed Feb 4, 2023
@ominousg ominousg removed the help wanted Extra attention is needed label Feb 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant