Skip to content

Node ready#2

Open
synthresin wants to merge 3 commits into
wonyoungso:masterfrom
synthresin:node-ready
Open

Node ready#2
synthresin wants to merge 3 commits into
wonyoungso:masterfrom
synthresin:node-ready

Conversation

@synthresin
Copy link
Copy Markdown

@synthresin synthresin commented May 1, 2016

npm 패키지로 기능할 수 있도록 Multilingual 객체를 다시 작성합니다.

1. package.json 정리

1-1. npm 권고에 맞게 패키지 이름에 .js 제거
1-2. 현재 당장 필요 없는 devDependency 제거. 정말 npm 상으로 필요할때 넣기.

2. index.js 작성하여, 해당 파일을 import 하여, Multilingual 객체 사용할 수 있게 하도록 함.

다른 webpack 기반 프로젝트에서 local npm 모듈로 로드하여, 잘 동작함을 확인함.

응용 코드 작성 사례
2016-05-01 4 56 21

결과
2016-05-01 4 56 10

3. 그외

prototype 자체 정의 대신, prototype 의 각 메소드를 정의하는 방식으로.(이게 사이드 이펙트가 없다나.)
패키지 이름과 맞추기 위해 MultiLingual 에서 Multilingual 로 객체 이름 변경.
변수 컨벤션 JS 에 맞게 camelCase 로.

4. 앞으로 해볼 만한 것.

이 npm 패키지를 사용하여 다른 응용 패키지를 만들수 있도록, 기능에서 DOM replace을 빼내기.
(이부분은 좀 이야기가 필요할 수도)
예시 코드 좀더 robust 하게. (이런데선 webpack 이 필요할지도)
npm 못쓰는 사람들에게 코드 제공하는법 찾아보기. (UMD build)

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.

1 participant