[Zzazan] 1주차 zzazanstagram 기능 구현 #4

wbluke · 2019-08-15T09:19:53Z

[구현한 기능 목록]

프론트엔드 뷰
회원가입
로그인
게시글 저장
게시글 조회

패키지 구조를 어떻게 가져가야 할지 잘 몰라서 많이 고민했습니다. 패키지 구조에 대해서 조언 부탁드려요 :)
리뷰 잘 부탁드립니다-! 감사합니다~

[#18] feat: 공통 패키지 및 엔티티 정의

1. Article 엔티티 클래스를 추가한다 2. Article 엔티티의 하위 VO 클래스를 추가한다 3. 그래들에 의존성을 추가한다

Feature/14 article frontend

Closes #8

Feature/6 member setting

1. 게시글을 작성하는 기능을 추가한다 1-1. ArticleController 클래스를 추가한다 1-2. POST 요청 시, 작성된 게시글을 저장한다 1-3. ArticleRequest DTO 클래스를 추가한다 1-4. ArticleRepository 클래스를 추가한다 2. 게시글 작성 기능에 대한 컨트롤러의 테스트 메서드를 추가한다

[#15] 게시글 작성 기능 구현 PR

[#26] refactor: 8월 13일 리뷰에 따른 리팩토링 적용

Closes #10

1. ArticleAssembler 클래스에 DTO로 변환하는 메서드를 추가한다 2. ArticleController에서 서비스 로직을 분리한다 3. 불필요한 어노테이션을 삭제한다

[#15] 게시글 조회 기능 구현 PR

Feature/6 signup frontend

kang-hyungu

코드 깔끔하게 잘 구현하셨어요 👍
패키지는 지금 구조로 가도 괜찮을 것 같네요
좀더 분리하자면 controller, (컨트롤러에서 사용하는)dto, member session 같이 표현 계층과 관련있는 패키지는 web으로 옮길 수도 있습니다.
몇 가지 코멘트 남겼으니 참고하시고 궁금한 점 있으면 코멘트 남기시거나 dm 주세요

kang-hyungu · 2019-08-16T12:51:34Z

src/main/java/com/woowacourse/zzazanstagram/model/article/domain/Article.java

+    @JoinColumn(name = "author", nullable = false, foreignKey = @ForeignKey(name = "fk_article_to_member"))
+    private Member author;
+
+    protected Article() {


상속을 허용하는게 아니라면 private 접근제어자를 추천드립니다.

하이버네이트에서 지연로딩을 구현할 때 엔티티 클래스를 상속해서 프록시 객체를 만든다고 알고 있습니다!
그래서 기본 생성자에 private을 지정해두게 되면 프록시 클래스에서 상속을 받지 못하기 때문에 protected로 두었습니다~

아 맞네요 그부분을 제가 놓쳤네요 👍

kang-hyungu · 2019-08-16T12:52:21Z

src/main/java/com/woowacourse/zzazanstagram/model/article/domain/Article.java

+        this.author = author;
+    }
+
+    public static Article from(final Image image, final Contents contents, final Member author) {


지금은 생성자로 충분해 보이는데 팩토리 메서드를 따로 만든 이유가 있을까요?

kang-hyungu · 2019-08-16T12:55:06Z

src/main/java/com/woowacourse/zzazanstagram/model/article/domain/vo/Contents.java

+import javax.persistence.Lob;
+
+@Embeddable
+public class Contents {


Contents를 클래스로 새로 정의할 필요가 있을까요?
Article 클래스에서 String contents 필드로 작성하면 코드가 더 간결해지지 않을까요?

Contents를 일급컬렉션으로 빼놓은 이유는 필드를 일관성 있게 구성하고 싶어서 였습니다!
만약 어떤 필드는 원시타입으로 관리하고 있는데 다른 필드가 일급컬렉션으로 관리하고 있다면 데이터를 관리할 때 혼란스러울 것이라고 생각했습니다 :D

일급컬렉션이요? wrapper class를 말씀하신걸까요?

데이터 관리할 때 어떤 점이 어려울지 잘 모르겠네요;
예시나 좀더 자세한 상황을 알려주실 수 있을까요?

kang-hyungu · 2019-08-16T12:56:03Z

src/main/java/com/woowacourse/zzazanstagram/model/article/domain/vo/Image.java

+
+    private String validateUrl(final String url) {
+        try {
+            HttpURLConnection.setFollowRedirects(false);


이미지가 http가 아닌 ftp에 있다면?
다른 통신 방법을 이용해야된다면 http에 의존하는 이 코드가 문제가 되지 않을까요?

이 부분은 저희가 차후에 S3를 사용해서 이미지 파일을 관리할 예정에 있기 때문에 ftp는 고려하지 않았습니다!

하지만 나중에 다른 기능을 구현하는 경우에는 이런 부분도 생각을 해봐야겠네요.
알려주셔서 감사합니다!

크롬이 이제 ftp 지원을 안한다고 합니다 ㅎㅎ; 그러니 저희도.. 😏

https://www.chromestatus.com/feature/6246151319715840

ftp는 예시구요, 도메인 코드에서 유효성 체크를 위해 http 통신을 할 필요가 있냐는 의견이에요
표현 계층에서 이미지가 존재하는지 결정해도 충분하지 않나요?
지금 이 유효성 체크는 이미지를 호출하는 외부 서비스가 응답이 없으면 같이 영향을 받습니다.
차라리 프론트에서 이미지를 못 불러오더라도 다른 기능은 동작하도록 하고, 이 객체는 이미지 위치만 전달하는 역할만 해도 충분해보입니다.

kang-hyungu · 2019-08-16T13:01:13Z

src/main/java/com/woowacourse/zzazanstagram/model/article/dto/ArticleRequest.java

+    private String image;
+
+    private String contents;
+    private String hashTag;


이 필드와 관련된 기능은 추가 예정인거죠? article 클래스에서 아직 관련 기능이 안보이네요 ㅎㅎ;

넵 :-D
도메인 설계는 거시적으로 생각하긴 했는데 한 번에 적용할 수는 없을 것 같아서 선언만 해놓았어요~

kang-hyungu · 2019-08-16T13:10:55Z

src/main/java/com/woowacourse/zzazanstagram/model/member/domain/vo/ProfileImage.java

+
+    private String validateUrl(final String url) {
+        try {
+            HttpURLConnection.setFollowRedirects(false);


인프라스트럭처 계층 코드가 있네요

kang-hyungu · 2019-08-16T13:12:10Z

src/main/java/com/woowacourse/zzazanstagram/model/member/domain/vo/ProfileImage.java

+        return url;
+    }
+
+    public ProfileImage updateUrl(String url) {


메서드명으로 update가 적절해보이진 않네요
valueOf 같이 불변 객체를 생성하는 의미가 담긴 메서드명을 찾아보는건 어떨까요

kang-hyungu · 2019-08-16T13:13:40Z

src/main/java/com/woowacourse/zzazanstagram/model/member/exception/MemberException.java

@@ -0,0 +1,10 @@
+package com.woowacourse.zzazanstagram.model.member.exception;
+
+public class MemberException extends IllegalArgumentException {


Member와 관련된 모든 오류를 이 예외에서 다 처리하지 말고 좀더 세분화해보는건 어떨까요?
회원 로그인 시도할 때 비밀번호가 틀렸을 때 예외, 회원을 찾을 수 없을 때 발생하는 예외 등등으로 나누면 에러 메시지에 의존하지 않고 어떤 이슈인지 빠르게 파악 가능할거에요

MemberException 세분화 잘 하셨어요 👍
그런데 MemberException이 IllegalArgumentException를 상속하는게 맞을까요?

kang-hyungu · 2019-08-16T13:16:22Z

src/main/java/com/woowacourse/zzazanstagram/model/member/service/MemberService.java

+        }
+    }
+
+    private void checkEnrolledNickName(String nickName) {


boolean existsByNickName(NickName nickName);
이 메서드를 memberRepository에 추가하면 코드가 좀더 간결해질거에요

kang-hyungu · 2019-08-16T13:19:56Z

src/test/java/com/woowacourse/zzazanstagram/model/article/domain/ArticleTest.java

+    }
+
+    @ParameterizedTest
+    @ValueSource(strings = {"https://image.shutterstock.com/image-photo/white-transparent-leaf-on-mirror-600w-1029171697.jpg", "https://image.shutterstock.com/image-photo/bright-spring-view-cameo-island-600w-1048185397.jpg"})


Suggested change

@ValueSource(strings = {"https://image.shutterstock.com/image-photo/white-transparent-leaf-on-mirror-600w-1029171697.jpg", "https://image.shutterstock.com/image-photo/bright-spring-view-cameo-island-600w-1048185397.jpg"})

@ValueSource(strings = {

"https://image.shutterstock.com/image-photo/white-transparent-leaf-on-mirror-600w-1029171697.jpg",

"https://image.shutterstock.com/image-photo/bright-spring-view-cameo-island-600w-1048185397.jpg"

})

IDE를 사용하시면 보이는 세로선은 이 margin까지만 코드를 작성하자는 컨벤션같은건데요.
옆으로 코드가 나열되면 가독성이 떨어집니다. 보통은 80 ~ 100이 기본입니다.

https://stackoverflow.com/questions/7756112/what-is-the-optimal-ide-text-editor-print-margin-when-programming

…d-#50 8/16 프론트엔드 리뷰 반영 리팩토링

dto에서 검사

[#55] refactor: 8/15 게시글 피드백 반영

[#52] refactor: 8/15 피드백 반영

woowahanCU

테스트코드 깔끔하게 잘 작성하셨네요! 💯
그리고 이슈도 잘 정리하고 있는 것으로 보이는데요.
칸반보드를 활용하지 않다보니 현재 진행중인 이슈는 무엇인지 확인하기가 어렵네요 ㅠㅠ
혹시 Projects 탭을 활용하지 않는 이유가 있을까요?

woowahanCU · 2019-08-17T07:00:15Z

build.gradle

+    mavenCentral()
+}
+
+dependencies {


gradlew 로 build 실행시에는 테스트를 skip하고 있어요.
아래 설정 추가한 이후에 배포해보시겠어요?

test { useJUnitPlatform() } ... dependencies { ... testImplementation 'org.junit.jupiter:junit-jupiter-api:5.3.1' testRuntimeOnly 'org.junit.jupiter:junit-jupiter-engine:5.3.1'

https://www.baeldung.com/junit-5-gradle

woowahanCU · 2019-08-17T07:02:15Z

src/main/java/com/woowacourse/zzazanstagram/model/article/domain/Article.java

+    }
+
+    public String image() {
+        return image.getUrl();


method name은 동작을 구체적으로 드러내는 동사를 사용하시는 것이 좋다고 생각합니다.

http://www.mimul.com/pebble/default/2013/05/04/1367639300999.html

woowahanCU · 2019-08-17T07:04:04Z

src/main/java/com/woowacourse/zzazanstagram/model/article/domain/vo/Contents.java

+import javax.persistence.Lob;
+
+@Embeddable
+public class Contents {


일급컬렉션이요? wrapper class를 말씀하신걸까요?

woowahanCU · 2019-08-17T07:06:30Z

src/main/java/com/woowacourse/zzazanstagram/model/article/domain/vo/Image.java

@@ -0,0 +1,44 @@
+package com.woowacourse.zzazanstagram.model.article.domain.vo;


model pacakge 하위에 controller, service, domain package가 있군요 🤔

woowahanCU · 2019-08-17T07:16:27Z

src/main/java/com/woowacourse/zzazanstagram/model/article/domain/vo/Image.java

+        try {
+            HttpURLConnection.setFollowRedirects(false);
+            HttpURLConnection con = (HttpURLConnection) new URL(url).openConnection();
+            con.setRequestMethod("HEAD");


Image class에서 URL 리소스의 유효성을 검증하는 것이 좋을까요?
Image model이 HttpURLConnection class와 연관관계를 가지는 것보다
Image 도메인 모델의 생애주기를 관리하는 repository의 구현체, 혹은 dao 등과 연관관계를 가지는 것은 어떨까요?

woowahanCU · 2019-08-17T07:27:40Z

src/main/java/com/woowacourse/zzazanstagram/model/member/domain/vo/Name.java

+        return name;
+    }
+
+    public Name updateName(String name) {


Name class이니, validateName, updatName 등의 Name도 중복이지 않을까요? 🙄

woowahanCU · 2019-08-17T07:28:22Z

src/main/java/com/woowacourse/zzazanstagram/model/member/domain/vo/ProfileImage.java

+        try {
+            HttpURLConnection.setFollowRedirects(false);
+            HttpURLConnection con = (HttpURLConnection) new URL(url).openConnection();
+            con.setRequestMethod("HEAD");


Image class에 있는 로직과 중복이네요..? 🙄

woowahanCU · 2019-08-17T07:30:27Z

src/main/java/com/woowacourse/zzazanstagram/model/member/service/MemberService.java

+
+    public void save(MemberSignUpRequest memberSignupRequest) {
+        checkEnrolledEmail(memberSignupRequest.getEmail());
+        checkEnrolledNickName(memberSignupRequest.getNickName());


매번 저장할때마다 조회쿼리가 여러번 발생하겠군요. 🙄

woowahanCU · 2019-08-17T07:31:21Z

src/test/java/com/woowacourse/zzazanstagram/model/RequestTemplate.java

+@ActiveProfiles("test")
+@AutoConfigureWebTestClient
+@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
+public abstract class RequestTemplate {


woowahanCU · 2019-08-17T07:33:01Z

src/test/java/com/woowacourse/zzazanstagram/model/article/domain/ArticleTest.java

+    }
+
+    @ParameterizedTest
+    @ValueSource(strings = {"https://image.shutterstock.com/image-photo/white-transparent-leaf-on-mirror-600w-1029171697.jpg", "https://image.shutterstock.com/image-photo/bright-spring-view-cameo-island-600w-1048185397.jpg"})


IDE를 사용하시면 보이는 세로선은 이 margin까지만 코드를 작성하자는 컨벤션같은건데요.
옆으로 코드가 나열되면 가독성이 떨어집니다. 보통은 80 ~ 100이 기본입니다.

https://stackoverflow.com/questions/7756112/what-is-the-optimal-ide-text-editor-print-margin-when-programming

wbluke · 2019-08-17T07:47:10Z

테스트코드 깔끔하게 잘 작성하셨네요! 💯
그리고 이슈도 잘 정리하고 있는 것으로 보이는데요.
칸반보드를 활용하지 않다보니 현재 진행중인 이슈는 무엇인지 확인하기가 어렵네요 ㅠㅠ
혹시 Projects 탭을 활용하지 않는 이유가 있을까요?

앗 내부적으로 ZenHub 사용해서 칸반보드 관리하고 있습니다! :) 기능이 많아서 좋은데 외부에선 보이지 않는다는 단점이 있네요ㅜㅜ

기존에 gradlew 로 build 실행시에는 테스트를 skip하고 있어서 https://www.baeldung.com/junit-5-gradle 를 참고하여 의존성 추가

기존의 existsByEmail, existsByNickName메서드를 통해서 각각 쿼리를 날리던 상황에서 existsByNickNameOrEmail를 사용하여 한 번만 쿼리를 날리도록 수정

Article, Member 클래스에서 Vo의 String 값을 요청하는 메서드명을 동사로 수정

wbluke · 2019-08-17T09:36:17Z

전반적으로 HTML5에서 제공하는 기본적인 form validation과 api를 매우 잘 활용하고 있습니다. 한 메서드의 길이도 길지 않고 깔끔하게 잘 구현하고 있어요~! 💯 😄
앞으로 기능이 추가되면서 자바스크립트 코드가 많아질텐데요~ 코드량이 늘어남에 따라 투두리스트 때 작성한 것처럼 구조화를 진행하면, 여러 크루들이 함께 협업하는데 더 큰 도움이 될 것 같아요.

리뷰 주셔서 감사합니다! 주신 피드백은 모두 적용하였는데 구조화 작업은 아직 자바스크립트 실력이 부족하여 진행하지 못하였습니다ㅠㅠ 공부를 조금 더 하고나서 적용해 보겠습니다...!

[#57] refactor: 8/17 씨유 코치님 리뷰 반영

[DEMO] ver.0.0.2 merge (0.0.1 refactor)

kang-hyungu

피드백 반영 잘 하셨어요 👍
몇 가지 추가 코멘트 남긴 건 참고하시고 다음 단계 진행하세요!

kang-hyungu · 2019-08-17T12:03:22Z

src/main/java/com/woowacourse/zzazanstagram/model/member/exception/MemberException.java

@@ -0,0 +1,10 @@
+package com.woowacourse.zzazanstagram.model.member.exception;
+
+public class MemberException extends IllegalArgumentException {


MemberException 세분화 잘 하셨어요 👍
그런데 MemberException이 IllegalArgumentException를 상속하는게 맞을까요?

woowahanCU

28e8bfb

어라, 이부분 추가안해줘도 테스트 동작하던가요? 🤔

test {
    useJUnitPlatform()
}

JunHoPark93 · 2019-08-17T13:26:45Z

28e8bfb

어라, 이부분 추가안해줘도 테스트 동작하던가요? 🤔
test {
    useJUnitPlatform()
}

앗 죄송합니다 빼먹었네요 ㅠ 다시 올려놓겠습니다

[#60] refactor: gradle test 추가

[DEMO] hotfix v.0.0.2

dasistHYOJIN

아주 멋있고 아름다운 코드와 구조네요
승인합니다

wbluke and others added 30 commits August 12, 2019 16:00

init: project 생성

20c71a5

init: 프론트 뷰 파일 초기 세팅

b535e2e

[#18] feat: 공통 패키지 및 엔티티 정의

572737a

Merge pull request #19 from wbluke/feature/18-common-package-setting

3b6501d

[#18] feat: 공통 패키지 및 엔티티 정의

[#15] feat: 게시글 모델 생성

bf13e9d

1. Article 엔티티 클래스를 추가한다 2. Article 엔티티의 하위 VO 클래스를 추가한다 3. 그래들에 의존성을 추가한다

[#7] feat: Member 엔티티 정의

1add6ef

[#20] refactor: index.html의 눈 모양 아이콘을 게시글 작성 아이콘으로 변경

8ae5520

[#20] feat: article을 작성하는 기본적인 페이지 작성

c42bb06

[#20] feat: article-edit.html의 자바스크립트 틀 작성

95b8923

Merge pull request #21 from wbluke/feature/14-article-frontend

ba1f455

Feature/14 article frontend

[#8] feat: 회원가입 기능

d975be9

Closes #8

Merge pull request #23 from wbluke/feature/6-member-setting

6ba9b74

Feature/6 member setting

Merge branch 'develop' into feature/14-article-create

8e9a725

Merge pull request #25 from wbluke/feature/14-article-create

e3a06c8

[#15] 게시글 작성 기능 구현 PR

[#22] refactor: 사용하지 않는 js console.log 삭제

6fbb985

[#26] refactor: 8월 13일 리뷰에 따른 리팩토링 적용

f6cef7a

Merge pull request #27 from wbluke/refactor/26-review

5f2ff75

[#26] refactor: 8월 13일 리뷰에 따른 리팩토링 적용

[#22] feat: zzazanstagram logo navBar에 추가 및 다른 페이지에도 적용

797d89e

[#29] refactor: 8월 13일 게시글 관련 리뷰에 따른 리팩토링 적용

a9bca9c

[#30] feat: 게시글 작성 페이지와 서버 시스템 연결

a53f71c

[#10] feat: 로그인 기능

9cf46ae

Closes #10

[#15] feat: 게시글 조회 기능 구현

f4ea6a1

1. ArticleAssembler 클래스에 DTO로 변환하는 메서드를 추가한다 2. ArticleController에서 서비스 로직을 분리한다 3. 불필요한 어노테이션을 삭제한다

[#31] feat: 게시글 조회 페이지와 서버 시스템 연결

e6a75cb

[#10] refactor: 테스트 중복 제거

6986556

[#22] feat: signup 페이지 뷰 작성

bcff14b

Merge branch 'develop' into feature/14-article-show

195db22

Merge pull request #35 from wbluke/feature/14-article-show

3eb2336

[#15] 게시글 조회 기능 구현 PR

Merge branch 'develop' into feature/6-signup-frontend

3d88698

Merge pull request #33 from wbluke/feature/6-signup-frontend

ffb423d

Feature/6 signup frontend

kang-hyungu requested changes Aug 16, 2019

View reviewed changes

wbluke and others added 7 commits August 17, 2019 11:07

Merge pull request #51 from wbluke/feature/14-review-refactor-fronten…

d8d4192

…d-#50 8/16 프론트엔드 리뷰 반영 리팩토링

[#52] refactor: 8/15 피드백 반영

c2ee81e

[#52] refactor: 8/15 피드백 반영

45e6b63

dto에서 검사

[#55] refactor: 8/15 게시글 피드백 반영

5caaace

Merge pull request #56 from wbluke/refactor/55-pr-review

cb34def

[#55] refactor: 8/15 게시글 피드백 반영

Merge branch 'develop' into refactor/52-pr-review

183d58e

Merge pull request #53 from wbluke/refactor/52-pr-review

e61f25f

[#52] refactor: 8/15 피드백 반영

woowahanCU approved these changes Aug 17, 2019

View reviewed changes

starkim06 added 7 commits August 17, 2019 17:09

[#57] refactor: 유틸 클래스(SessionKeys) 생성자 private 으로 선언

cd492bb

[#57] feat: test 용 의존성 추가

28e8bfb

기존에 gradlew 로 build 실행시에는 테스트를 skip하고 있어서 https://www.baeldung.com/junit-5-gradle 를 참고하여 의존성 추가

[#57] refactor: Image 클래스에서 Http요청을 통해서 이미지 URL확인로직 제거

ecaf16d

[#57] refactor: RequestMapping을 활용하여 중복 Url제거

10d1acd

[#57] refactor: validate 메서드 명 수정

21ce84b

[#57] refactor: Email, NickName에 대해서 각각 요청하던 쿼리를 합침

96499af

기존의 existsByEmail, existsByNickName메서드를 통해서 각각 쿼리를 날리던 상황에서 existsByNickNameOrEmail를 사용하여 한 번만 쿼리를 날리도록 수정

[#57] refactor: 메서드명 동사로 수정

9ea9d2c

Article, Member 클래스에서 Vo의 String 값을 요청하는 메서드명을 동사로 수정

wbluke and others added 3 commits August 17, 2019 18:55

[#57] refactor: SessionKeys를 web package로 이동

6619430

Merge pull request #58 from wbluke/refactor/57-review

243a54e

[#57] refactor: 8/17 씨유 코치님 리뷰 반영

Merge pull request #59 from wbluke/develop

276e3a8

[DEMO] ver.0.0.2 merge (0.0.1 refactor)

kang-hyungu approved these changes Aug 17, 2019

View reviewed changes

woowahanCU reviewed Aug 17, 2019

View reviewed changes

JunHoPark93 and others added 3 commits August 17, 2019 22:27

[#60] refactor: gradle test 추가

297f7d6

Merge pull request #61 from wbluke/refactor/60-gradle-test

13bf0ce

[#60] refactor: gradle test 추가

Merge pull request #62 from wbluke/develop

9fef616

[DEMO] hotfix v.0.0.2

wmakerjun approved these changes Aug 19, 2019

View reviewed changes

dasistHYOJIN approved these changes Aug 19, 2019

View reviewed changes

kang-hyungu merged commit 7cf70ba into woowacourse:zzazan Aug 19, 2019

		@@ -0,0 +1,10 @@
		package com.woowacourse.zzazanstagram.model.member.exception;

		public class MemberException extends IllegalArgumentException {

		@@ -0,0 +1,44 @@
		package com.woowacourse.zzazanstagram.model.article.domain.vo;

[Zzazan] 1주차 zzazanstagram 기능 구현 #4

[Zzazan] 1주차 zzazanstagram 기능 구현 #4

Conversation

wbluke commented Aug 15, 2019

kang-hyungu left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

woowahanCU Aug 17, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

woowahanCU left a comment

Choose a reason for hiding this comment

woowahanCU Aug 17, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

wbluke commented Aug 17, 2019

wbluke commented Aug 17, 2019

kang-hyungu left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

woowahanCU left a comment

Choose a reason for hiding this comment

JunHoPark93 commented Aug 17, 2019

dasistHYOJIN left a comment

Choose a reason for hiding this comment

woowahanCU Aug 17, 2019 •

edited

Loading

woowahanCU Aug 17, 2019 •

edited

Loading