CODE SQUAD/FeedBack 정리25 [Review] 2022/06/23 마지막 PR (Issue Tracker) 이번 리뷰는 Brain 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/issue-tracker/pull/200 [Team-26][BE : Shine] Issue-Tracker 2주차 2회차 PR by zbqmgldjfh · Pull Request #200 · codesquad-members-2022/issue-tracke 안녕하세요 Shine입니다!! 저의 리뷰를 담당해주시게 되어 감사하다는 말씀 전하고 싶습니다. 질문 3가지 우선 로직상의 질문이 아닌, 사용법 에 관한 질문을 드리게 되어 유감스럽게 생각합니다 github.com https://github.com/codesquad-members-2022/issue-tracker/p.. CODE SQUAD/FeedBack 정리 2022. 7. 5. [Review] 2022/06/23 3차 PR (Issue Tracker) 이번 리뷰는 Brain 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/issue-tracker/pull/124#issuecomment-1163833790 [Team-26][BE : Shine] Issue-Tracker 2주차 1회차 PR by zbqmgldjfh · Pull Request #124 · codesquad-members-2022/issue-tracke 안녕하세요! Shine 입니다! 저의 리뷰를 담당해주시게 되어 감사하다는 말씀 전하고 싶습니다. 질문 1가지 1. API에서의 Redirction은? Client 측에서 POST로 자원 생성 요청이 올 경우, Server에서 사용자가 github.com 1. 질문 과연 A.. CODE SQUAD/FeedBack 정리 2022. 6. 24. [Review] 2022/06/21 2차 PR (Issue Tracker) 이번 리뷰는 Brain 께서 해주셨다! 리뷰해주셔서 감사합니다 !! [Team-26][BE : Shine] Issue-Tracker 1주차 2회차 PR by zbqmgldjfh · Pull Request #94 · codesquad-members-2022/issue-tracker 안녕하세요! Shine 입니다! 우선 저의 리뷰를 담당해주시게 되어 감사하다는 말씀 먼저 전하게 됩니다. 질문 2가지 1. DTO 디렉토리의 위치 DTO를 controller → service 로 넘길때도 사용하고, service → con github.com 1. 코드리뷰 1-1) JWT token 응답은 body로! 원래 나는 사용자가 로그인 하면 응답으로 JWT token을 Header에 담아서 보내고 있었다. 리뷰어의 말.. CODE SQUAD/FeedBack 정리 2022. 6. 21. [Review] 2022/06/09 5차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! [Team - 06][BE] 에어비앤비 프로젝트 3주차 1회차 PR by leejohy-0223 · Pull Request #237 · codesquad-members-2022/a 안녕하세요 Dion! 매번 저희 팀의 리뷰를 담당해 주셔서 감사의 인사를 전합니다!! 1. 구현 사항 이번에는 API를 구분하여 각각 구현해보았고, 추가된 부분은 다음과 같습니다. Wish API 구현 Reservation github.com 1. 코드리뷰 1-1) 환경 변수에 대한 고민 내가 스크립트를 짜면서도 Dion과 같은 생각을 했다... 이걸 항상 다 설정해줘야 하나?? .... 이에 대한 고민은 너무 길어지는 것 같아 따로 글로 작성하였다. https.. CODE SQUAD/FeedBack 정리 2022. 6. 12. [Review] 2022/06/04 4차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/airbnb/pull/203#discussion_r891396646 [Team - 06][BE] 에어비앤비 프로젝트 2주차 2회차 PR by zbqmgldjfh · Pull Request #203 · codesquad-members-2022/air 안녕하세요 Dion! 우선 저희 팀의 리뷰를 담당해 주셔서 감사의 인사를 전합니다!! 질문이 한가지 있습니다! 구현 사항 이번에 구현한 사항은 다음과 같습니다. GitHub, Kakao OAuth Login 구현 JWT Token 기 github.com 1. 질문 1-1) URL의 설계 개발을 진행하던 중 URL설계에 대.. CODE SQUAD/FeedBack 정리 2022. 6. 9. [Review] 2022/06/01 3차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/airbnb/pull/140 [Team - 06][BE] 에어비앤비 프로젝트 2주차 1회차 PR by leejohy-0223 · Pull Request #140 · codesquad-members-2022/a 안녕하세요 Dion! 우선 저희 팀의 리뷰를 담당해 주셔서 감사의 인사를 전합니다!! 주요 변경사항으로는 Native Query를 JPQL로 변경한 부분입니다. 수정사항이 많지는 않습니다만, 한 가지 질문이 있 github.com 1. 코드 리뷰 1-1) Spring Actuator 리뷰어 께서 Spring Actuator에 대하여 학습해보길 권해주셨다! 본질.. CODE SQUAD/FeedBack 정리 2022. 6. 4. [Review] 2022/05/27 2차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/airbnb/pull/88 [Team - 06][BE] 에어비앤비 프로젝트 1주차 2회차 PR by zbqmgldjfh · Pull Request #88 · codesquad-members-2022/airb 안녕하세요 Dion! 우선 저희 팀의 리뷰를 담당해 주셔서 감사의 인사를 전합니다!! 이번 2차 PR을 진행하면서 한가지 고민거리가 있었는데... 이에 대하여 답변해주시면 감사하겠습니다 ㅠ,ㅠ 1. Nat github.com 1. 질문 Native Query 질문 다음 두 메서드 searchByCondition 과 searchByConditionQueryDsl .. CODE SQUAD/FeedBack 정리 2022. 6. 1. [Review] 2022/05/25 1차 PR (Aribnb 서비스) 이번 리뷰는 Dion 께서 해주셨다! 리뷰해주셔서 감사합니다 !! https://github.com/codesquad-members-2022/airbnb/pull/40#discussion_r883130404 [Team - 06][BE] 에어비앤비 프로젝트 1주차 1회차 PR by leejohy-0223 · Pull Request #40 · codesquad-members-2022/ai 안녕하세요 Dion! Shine & Lucid 입니다! 1주차 1회차 PR 리뷰 요청드립니다. 주요 구현 사항 논리적 모델링 물리적 모델링 엔티티 생성 및 연관관계 형성 질문 사항 Shine 1.위치 검색에 대한 질문 사용 github.com 1. 질문 1-1) 위치 검색에 대한 질문 사용자의 위치(위도, 경도)를 중심으로.. CODE SQUAD/FeedBack 정리 2022. 5. 27. [Review] 2022/04/22 2차 PR (반찬 서비스) 이번 리뷰는 wooody 께서 해주셨다! 리뷰해주셔서 감사합니다 !! [Team-12 BE] 1주차 두번째 PR (2022/4/22) by geombong · Pull Request #106 · codesquad-members-2022/sidedish 안녕하세요 리뷰어님 팀12 백엔드 샤인&검봉 입니다. 저희 팀의 리뷰를 담당해주셔서 우선 감사의 말 전합니다! 1차 PR 리뷰 수정사항 categoryService 에서 pageId 받는 부분 수정하기 schema 처음과 끝 f. github.com 1. 질문 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 1) 요구사항에 없는 아이템 등록(상품 등록) 같은 기능 또한 고려하여 DB 설계를 진행해야 하는지 궁금합니다. 2) .. CODE SQUAD/FeedBack 정리 2022. 4. 27. [Review] 2022/04/20 1차 PR (반찬 서비스) 이번 리뷰는 wooody 께서 해주셨다! 리뷰해주셔서 감사합니다 !! 1. 질문 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 팀원과 Spring Data Jdbc를 사용하던 도중, OneToMany 관계를 이용할 때 Map을 통해 저장하도록 하였었습니다. Category : item = 1 : N 의 관계로 말이죠! 하지만 해당 Map을 이용할 경우 어그리게이트 루트에 해당되는 Category는 auto_increment가 잘 적용되지만, item은 auto_increment가 정상작동하지 않았습니다. 혹시 Map을 사용할때는 id값을 직접 지정해줘야 하나요? @Test public void itemSaveTest() { // given Category mainCate.. CODE SQUAD/FeedBack 정리 2022. 4. 21. [Review] 2022/04/10 2차 PR 1. 코드 리뷰 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 1. 기존 순서와 겹치는 경우 순서가 겹치는 경우 해당 1번이 뒤로 밀리고, 그 사이에 삽입 되도록 하였다. 원래의 순서 1번은 2번이 되게 될 것 이다! 2. 초기화 메서드 추출하기 초기화에 해당되는 부분이다. 위 코드는 생성자에서 한번 호출되기 때문에 사실 따로 분리시키지는 않았었다. 리뷰어님의 말씀처럼 메서드로 뽑을 수 도 있지만, 그냥 명시적으로 생성자 안에서 초기화 하고 있음을 보이는 것 또한 좋다 생각한다! 3. 로직을 함수로 더 분리하기 함수 안에 여러 로직들이 퍼져 있다. 관련된 하나의 덩어리 들 끼리 메서드로 뽑아 주어야 할것 같다. CODE SQUAD/FeedBack 정리 2022. 4. 14. [Review] 2022/04/06 1차 PR 1. 질문 " data-ke-type="html"> HTML 삽입 미리보기할 수 없는 소스 팀원과 처음 Work라는 도메인 모델을 만들때 2가지 방식의 생성방법에 대하여 논의 하게 되었습니다. 생성자 방식 빌더패턴 1) 생성자 이용하기 public class Work { // 생략... public Work(String title, String content, String author) { this.title = title; this.content = content; this.author = author; this.workStatus = WorkStatus.TODO; // 초기값 자동 지정 this.createDateTime = LocalDateTime.now(); // 초기값 자동 지정 this.las.. CODE SQUAD/FeedBack 정리 2022. 4. 10. 이전 1 2 3 다음
