keystonejs/keystone-classic

Keystone admin JS not minified

Open

#2.825 geöffnet am 12. Mai 2016

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (14.656 Stars) (2.288 Forks)batch import
help wanted

Beschreibung

http://localhost:3000/keystone/js/index.js is not minified.

$ NODE_ENV=production node server.js 
$ curl http://localhost:3000/keystone/js/index.js | wc -c
 1312064 (1.3M)
$ curl http://localhost:3000/keystone/js/index.js | uglify -cm | wc
  472863 (473K)

Contributor Guide

Tech Stack
javascriptnodejsexpress
Domain
performance
Issue Type
performance
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Keine
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80
Research-Richtung
Look into the build configuration file (likely gulpfile.js or Gruntfile.js) to add a minification step for the JavaScript bundle. Consider using uglify js or similar. There are no linked PRs or comments indicating prior work. The admin JS is served at /keystone/js/index.js, so the fix involves adding a minification plugin in the production build pipeline.