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

KOMORANAdmin을 KOMORAN repository로 통합 #106

Merged
merged 100 commits into from
Apr 6, 2020
Merged

KOMORANAdmin을 KOMORAN repository로 통합 #106

merged 100 commits into from
Apr 6, 2020

Conversation

shin285
Copy link
Member

@shin285 shin285 commented Apr 3, 2020

관련 이슈 또는 PR 번호

Resolve #104

PR 종류

  • 신규 기능

PR 설명

  • 기존 KOMORAN을 core 모듈로 변경
  • KOMORANAdmin을 admin 모듈로 통합

@shin285 shin285 requested review from gamamoe and 9bow and removed request for gamamoe and 9bow April 3, 2020 14:02
@shin285
Copy link
Member Author

shin285 commented Apr 3, 2020

@9bow @dolpang2
KOMORANAdmin을 통합함에 따라 프로젝트 구조가 변경되었습니다.
KOMORAN 프로젝트 밑에 core 모듈(기존 KOMORAN), admin 모듈(기존 KOMORANAdmin)이 위치하는 구조입니다.
추후 elasticsearch plugin이나 #97 과 같은 내용들은 KOMORAN 프로젝트에 모듈 단위로 추가하면 될 것 같습니다.

@9bow 9bow requested review from 9bow and gamamoe April 3, 2020 15:01
@gamamoe
Copy link
Member

gamamoe commented Apr 4, 2020

일요일까지 보고 코멘트 남기겠습니당

Comment on lines +21 to +22
include 'admin'
include 'admin'
Copy link
Member

Choose a reason for hiding this comment

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

admin 모듈이 한 번만 추가되어도 될 것 같습니다.

Suggested change
include 'admin'
include 'admin'
include 'admin'

Copy link
Member Author

Choose a reason for hiding this comment

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

merge 후에 진행하겠습니다.


dependencies {
// use latest KOMORAN
compile 'com.github.shin285:KOMORAN:3.3.8'
Copy link
Member

Choose a reason for hiding this comment

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

이 부분은 merge 후 추가 수정을 하는 것으로 @shin285 님께서 말씀해주셨습니다.

Copy link
Member

@9bow 9bow left a comment

Choose a reason for hiding this comment

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

매뉴얼 및 KOMORAN 참고 방법 등, 추가 수정할 부분들은 merge 후 진행하도록 하겠습니다.

@shin285 shin285 merged commit 490c9c7 into master Apr 6, 2020
@shin285 shin285 deleted the issue-104 branch April 14, 2020 15:18
@shin285 shin285 restored the issue-104 branch April 15, 2020 15:00
shin285 added a commit that referenced this pull request Apr 15, 2020
shin285 added a commit that referenced this pull request Apr 15, 2020
@shin285 shin285 deleted the issue-104 branch April 15, 2020 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

KOMORANAdmin을 KOMORAN repository로 통합
3 participants