keystonejs/keystone-classic

Keystone admin JS not minified

Open

#2 825 ouverte le 12 mai 2016

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (14 656 stars) (2 288 forks)batch import
help wanted

Description

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)

Guide contributeur

Stack technique
javascriptnodejsexpress
Domaine
performance
Type d'issue
performance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
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.