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

Step4 - [4단계 - multi module 적용] #58

Merged
merged 2 commits into from
Jan 23, 2021
Merged

Conversation

jhhj424
Copy link

@jhhj424 jhhj424 commented Jan 21, 2021

안녕하세요 시홍님!

마지막 미션, 4단계 - multi module 적용 완료하여 PR 드립니다!!


사실 처음 이번 미션을 시작할때 막연한 막막함이 너무 컸었습니다.
그래서 미션에 의도부터 시홍님께 물어보고 그랬었죠...

시홍님께 피드백 받는 과정동안 저도 많은 학습을 할 수 있었고 발전할 수 있었다고 생각합니다.

이번 미션간 긴 시간은 아니었지만, 시홍님께 많은 것을 배우고 얻어가는 것 같습니다.
덕분에 더 많은 고민도 할 수 있었구요 :)

아직도 갈길이 구만리지만요 ㅎㅎ 🤣🤣

어쨌든 무엇보다 재미있는 시간이었던 것 같습니다!! 덕분입니다, 감사합니다🙏


마지막 4단계도 잘 부탁드리겠습니다🙏🙏

Copy link

@hongsii hongsii left a comment

Choose a reason for hiding this comment

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

4단계 요구사항 구현도 잘 해주셨네요!
간단한 피드백 남겼으니 확인하셔서 반영해주세요~!

build.gradle Outdated Show resolved Hide resolved
build.gradle Outdated Show resolved Hide resolved
build.gradle Outdated
useJUnitPlatform()
project(':module-domain') {
dependencies {
compile project(':module-common')
Copy link

Choose a reason for hiding this comment

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

gradle 버전이 올라가면서 compile은 deprecated 되었습니다.
아래의 글을 읽어 보고 compile을 다른 걸로 바꿔보면 어떨까요?

Copy link
Author

Choose a reason for hiding this comment

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

그렇군요!! 좋은 참고 자료 주셔서 감사합니다 :)
그런데 저는 api 사용이 잘 안되는거 같은데 이유가 있을까요 ㅠ
implementation 을 사용할 수 있도록 했습니다.

Copy link

Choose a reason for hiding this comment

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

api는 추가 플러그인을 설정해주면 됩니다.

Copy link
Author

Choose a reason for hiding this comment

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

감사합니다 시홍님👍👍

1. common 모듈 제거
2. application 모듈 제거
3. 실행하는 애플리케이션마다 비즈니스 로직이 달라질 수 있다, 조립 가능한 단위로 현재는 `web application` 만 사용하므로 `app-web` 모듈로 통합&분리
Copy link

@hongsii hongsii left a comment

Choose a reason for hiding this comment

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

피드백 반영을 깔끔하게 잘 해주셨네요! 💯
답변 남겼으니 확인해주시고 이번 미션은 여기서 마무리할게요.
미션 진행하시느라 고생 많으셨습니다~! 🎉
마지막까지 힘내셔서 마무리 잘하세요!

@hongsii hongsii merged commit d9f4cdc into next-step:jhhj424 Jan 23, 2021
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.

None yet

2 participants