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

To add chat.js in the all.js (minify) #177

Closed
codenjoyme opened this issue Aug 16, 2021 · 5 comments
Closed

To add chat.js in the all.js (minify) #177

codenjoyme opened this issue Aug 16, 2021 · 5 comments

Comments

@codenjoyme
Copy link
Owner

У нас есть maven minify плагин, который позволяет сжимать js. Я пробовал использовать ES6 фичи в коденжое (в одном месте в chat.js), и этот плагин не позволял Сжимать скрипт с промисами. Потому родился вот этот патч

https://github.com/codenjoyme/minify-maven-plugin/tree/es6-support

Мы предлагали его PR для автора, не знаю принял ли он или нет. Надо убедиться что его приняли, и если нет настоять на этом (получить новую версию или отказ аргументированный). Если не войдет патч, тогда разместить на нашем codenjoy-repo артефакт.

Дальше взять за основу это решение и сжать весь скрипт в all.js

Сейчас скрипт

https://github.com/codenjoyme/codenjoy/blob/develop/CodingDojo/server/src/main/webapp/resources/js/chat.js

не входит в all.js из за того что описал выше и сам по себе, что увеличивает энтропию

Вот тут он отдельно добавляется

https://github.com/codenjoyme/codenjoy/blob/develop/CodingDojo/server/src/main/webapp/view/common-inclusion.jsp#L41

И вот тут он не включен, а должен бы

https://github.com/codenjoyme/codenjoy/blob/develop/CodingDojo/server/pom.xml#L1067

@codenjoyme
Copy link
Owner Author

048f90f

@codenjoyme
Copy link
Owner Author

@codenjoyme
Copy link
Owner Author

codenjoyme commented Aug 20, 2021

https://github.com/codenjoyme/codenjoy/blob/develop/CodingDojo/server/src/main/webapp/view/common-inclusion.jsp#L41
тут надо убрать еще может быть еще где-то подключен, по прокту поиском надо посмотреть

@codenjoyme
Copy link
Owner Author

12e2a30

@codenjoyme
Copy link
Owner Author

@codenjoyme codenjoyme added this to the v1.1.1 milestone Dec 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants