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

Grunt 대신 WebPack 도입 #1042

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

lshqqytiger
Copy link
Contributor

@lshqqytiger lshqqytiger commented Sep 3, 2022

grunt는 최신 자바스크립트 문법 일부를 지원하지 않아 프론트엔드 개발 시 불편합니다.
따라서 Grunt를 WebPack으로 교체하여 최신 자바스크립트 문법을 이용하고 동시에 용량이 큰 인 게임 빌드 결과물을 경량화할 것을 제안합니다.
grunt default pack 대신 npm run build 또는 npx webpack으로 배포합니다.

global.min.js 4.73 KB → 4.85 KB
in_admin.min.js 11 KB → 11.0 KB
in_game_kkutu.min.js 99.1 KB87.3 KB
in_login.min.js 737 Bytes → 764 Bytes
in_portal.min.js 2.17 KB → 2.22 KB

※ 인 게임 플레이 정상 작동까지 확인되었으나 다른 부분에 문제가 있을 수 있습니다.
그런 경우 Comment 남겨주시면 확인하겠습니다.

@norhu1130
Copy link

본 PR을 적용하였으나, Portal에서 이런 오류가 발생하며 정상적인 화면이 표시되지 않네요.

global.min.js:17 Uncaught TypeError: Cannot read properties of undefined (reading 'LOGIN')
    at HTMLDocument.<anonymous> (global.min.js:17:2908)
    at f (jquery.js:19:18299)
    at Object.fireWith [as resolveWith] (jquery.js:19:19085)
    at Function.ready (jquery.js:19:13909)
    at HTMLDocument.xt (jquery.js:19:11409)

image
global.js 125번쨰줄에 L['LOGIN']을 "로그인" 으로 바꾸면 정상적으로 표시됩니다.
ko_KR.json에는 정상적으로 해당 값이 있습니다.

@norhu1130
Copy link

본 PR을 적용하였으나, Portal에서 이런 오류가 발생하며 정상적인 화면이 표시되지 않네요.

global.min.js:17 Uncaught TypeError: Cannot read properties of undefined (reading 'LOGIN')
    at HTMLDocument.<anonymous> (global.min.js:17:2908)
    at f (jquery.js:19:18299)
    at Object.fireWith [as resolveWith] (jquery.js:19:19085)
    at Function.ready (jquery.js:19:13909)
    at HTMLDocument.xt (jquery.js:19:11409)

image global.js 125번쨰줄에 L['LOGIN']을 "로그인" 으로 바꾸면 정상적으로 표시됩니다. ko_KR.json에는 정상적으로 해당 값이 있습니다.

확인을 위해 본 PR에 포함된 global.min.js를 적용하였으나 동일한 문제가 있습니다.

@lshqqytiger
Copy link
Contributor Author

수정했습니다.

@Kimu-Nowchira
Copy link

적용해 테스트해본 결과 작동에 이상은 없는 것 같습니다.

Copy link
Contributor

@kdhkr kdhkr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

기능상의 작동에는 문제가 없어 보입니다.

@xxl0u
Copy link

xxl0u commented Nov 19, 2022

image
적용하고 npm run build 돌려보니 이렇게 뜨네요.

@lshqqytiger
Copy link
Contributor Author

Server/lib/Web/lib/in_game_kkutu.js 파일이 없는 경우 발생하는 문제로 확인되었습니다.
ffd8712에서 해결되었습니다.

@xxl0u
Copy link

xxl0u commented Nov 19, 2022

Server/lib/Web/lib/in_game_kkutu.js 파일이 없는 경우 발생하는 문제로 확인되었습니다. ffd8712에서 해결되었습니다.

(ffd8712)
적용 후 작동 잘 됩니다. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants