You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Como compilar los assets no está documentado y el directorio de assets está algo desordenado.
Por ejemplo:
Hay archivos como boostrap.min.css y multiple-select.min.css que son archivos mitificados, donde el primero parece que no se usa y ambos son dependencias de node.
Hay scripts que podrían simplificarse y ordenarse demasiado utilizando JavaScript moderno o utilizando más funciones, como en la creación de objetos en query_body de search.js y en la única función de 97 líneas con ~4 funciones anónimas anidadas en quota.js.
Al correr webpack se genera un directorio dist que no está completamente ignorado por git.
Ver para que es cada archivo requiere buscar como se usan en webpack.config.js y los otros scripts.
Entonces, para mejorar el orden y facilitar la resolución de #4, #15, #20, #21 y #35 creo que es necesario:
Documentar como generar (o como funcionan) los assets
Agilizar la generación de los assets
Crear un directorio de como helpers o utils y dejar ahí los scripts auxiliares (los que no están en entry de webpack)
Eliminar los archivos minificados que podrían importarse de una dependencia de node
Eliminar dist del repositorio e ignorarlo
Una vez visto eso, se podría además ver si:
Hacer un refactor de algunas partes del código para hacer más ágil los cambios a futuro
Dejar un linter o formatter como eslint
Se elimina la dependencia de jquery
Sobreescribir los colores principales de Boostrap en base.css
The text was updated successfully, but these errors were encountered:
Toda la carpeta de assets es tan desastrosa como mi entendimiento sobre webpack, js moderno y css moderno.
Un buen punto de partida podría ser trasladar algunos archivos a un directorio utils (como dices, los que no son entries), eliminar los archivos que puedan importarse como dependencias, hacer documentación e incorporar un formatter.
Preferiría mantener la carpeta dist mientras que no esté automatizada la compilación de assets, para no complejizar más el setup y deploy por mientras.
Faltó ver el tema de agilizar la generación de assets, idealmente se deberían generar al haber cambios cuando se corre el servidor al probarlo. Pero por lo que he visto, django no tiene una buena integración con webpack, entonces se podría dejar como algo bueno a tener en el futuro y no necesario.
Como se cumplió la meta de la issue, la cierro nomás.
Como compilar los assets no está documentado y el directorio de assets está algo desordenado.
Por ejemplo:
boostrap.min.css
ymultiple-select.min.css
que son archivos mitificados, donde el primero parece que no se usa y ambos son dependencias denode
.query_body
desearch.js
y en la única función de 97 líneas con ~4 funciones anónimas anidadas enquota.js
.webpack
se genera un directoriodist
que no está completamente ignorado por git.webpack.config.js
y los otros scripts.Entonces, para mejorar el orden y facilitar la resolución de #4, #15, #20, #21 y #35 creo que es necesario:
helpers
outils
y dejar ahí los scripts auxiliares (los que no están enentry
dewebpack
)node
dist
del repositorio e ignorarloUna vez visto eso, se podría además ver si:
eslint
jquery
base.css
The text was updated successfully, but these errors were encountered: