keystonejs/keystone-classic

Keystone admin JS not minified

Open

#2.825 aberto em 12 de mai. de 2016

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)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)

Guia do colaborador

Pilha de tecnologia
javascriptnodejsexpress
Domain
performance
Tipo Issue
performance
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
None
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80
Direção de pesquisa
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.