[Team-16][BE][Yanse] AirBnB 프로젝트 1주차 PR#1 #8

rkolx · 2022-05-25T07:42:36Z

안녕하세요! Henry! Yanse(Yan & Hanse)입니다
첫 PR은 간단하게 프로젝트 세팅 및 엔티티 설계를 진행하였습니다!
앞으로 삼주간 잘 부탁드리겠습니다!🙇🏻

초기 ERD

진행완료

요구사항 분서
테이블 설계
API 설계 초안

감사합니다🙇🏻

파트 폴더 생성

closes_#2

- 연관관계 기본 설정 완료 closes_#3

feat: 엔티티 만들기

closes_#5

chore: gitignore 추가

- 연관관계 매핑 수정 - room에 이미지 컬럼추가 및 main 도메인 생성 closes_#8

refactor: 엔티티 수정및 추가

chore: 환경변수 설정

wooody92

안녕하세요. 우선 개발 진행하느라 수고 많으셨습니다 !!
초기 설계 단계라서 엔티티 관련하여 코멘트 남겼습니다. 추가로 ERD 관련하여 조금 더 부연 설명이 있었으면 더 빠르게 파악하기 좋았을 것 같아요~ (다음 PR에도 ERD 링크는 계속 걸어주세요.)
남은 기간도 화이팅입니다. 💪

wooody92 · 2022-05-25T09:31:21Z

BE/src/main/java/yanse/airbnb/domain/main/MainRecommendationImage.java

+@Builder
+@AllArgsConstructor
+@NoArgsConstructor(access = AccessLevel.PROTECTED)


(코드 전반적으로) 초기 설계 단계에서 무조건적으로 오픈해놓고 시작하는 건 좋은 습관은 아닙니다. 객체 생성이 필요한 경우에만 사용해주세요..

넵! 필요할때만 생성하는걸로 하겠습니다:)

wooody92 · 2022-05-25T09:33:34Z

BE/src/main/java/yanse/airbnb/domain/hotel/Room.java

+@AllArgsConstructor
+@NoArgsConstructor(access = AccessLevel.PROTECTED)


전체필드 생성자는 별도의 접근 제한이없고, 기본 생성자만 접근제한 레벨 설정한 이유가 있나요?

전체필드는 처음 생성시 해당 애노테이션을 다른클래스에 붙여넣기를 하는 과정에서 모두 동일하게 되었습니다 해당 부분 신경쓰고 수정하겠습니다!

wooody92 · 2022-05-25T09:35:59Z

BE/src/main/java/yanse/airbnb/domain/hotel/Room.java

@@ -0,0 +1,50 @@
+package yanse.airbnb.domain.hotel;


아래처럼 패키지도 패키지의 main table에 맞추는 것이 일관성있어 보입니다.

Suggested change

package yanse.airbnb.domain.hotel;

package yanse.airbnb.domain.room;

클래스명 수정하면서 패키지명을 따로 신경쓰지 못했습니다ㅠ^ㅠ 해당 부분도 신경써서 수정하도록 하겠습니다!

wooody92 · 2022-05-25T09:45:37Z

BE/src/main/java/yanse/airbnb/domain/hotel/Room.java

+
+	private double stars;
+
+	private int review;


이건 리뷰 갯수를 의미하는게 맞다면 review_count 네이밍은 어떤가요?
요구사항 밖으로 알고있지만 Room - Review 관계로 있고 Review 내부에 user, comment 등등 필드가 있도록 확장하는 경우가 있을 것 같아서요.

해당 네이밍이 더 명확해보입니다! 앞으로도 좀 더 명확학 네이밍을 사용하도록 노력하겠습니당!

wooody92 · 2022-05-25T09:53:01Z

BE/src/main/java/yanse/airbnb/domain/hotel/Room.java

+	@Column(columnDefinition = "TEXT")
+	private String description;
+
+	private double stars;


이건 평점 필드로 보이네요. 뭔가 rate 또는 rating이라는 네이밍을 활용하면 좋을 것 같은 생각도 듭니다. 개인적으로는 특수한 케이스가 아니면 복수형 이름은 아래와 같은 상황때문에 지양하는 편입니다.

평점 목록 정보가 필요한 경우 네이밍을 어떻게 가져가야하는지..? starsList, stars 둘다 어색..

해당부분도 위와같이 네이밍에 신경써서 구현하도록 하겠습니다!

wooody92 · 2022-05-25T09:54:42Z

BE/src/main/java/yanse/airbnb/domain/hotel/RoomInfo.java

+@Embeddable
+public class RoomInfo {
+
+	private String roomType;


type 관련 필드는 enum으로 관리하는게 좋을 것 같습니다.

type관련 필드들이 나올경우 enum으로 관리해보도록 하겠습니다!

wooody92 · 2022-05-25T09:56:20Z

BE/src/main/java/yanse/airbnb/domain/main/MainAroundImage.java

+@AllArgsConstructor
+@NoArgsConstructor(access = AccessLevel.PROTECTED)
+@Entity
+public class MainAroundImage {


이건 어떤 데이터를 담기위한 엔티티일까요?

주변 여행지와 관련된 이미지 엔티티인데 해당 부분은 한 이미지 테이블에 타입으로 나눌 수 있도록 변경 할 예정입니다:-)

wooody92 · 2022-05-25T09:58:51Z

BE/src/main/java/yanse/airbnb/domain/reservation/Reservation.java

+	private Room room;
+
+	@Column(updatable = false)
+	private int reservationPeople;


숙박인원 수 인가요? 네이밍 조금 더 고민해주시면 좋을 것 같아요.

해당 부분 고민해보고 수정해보도록 하겠습니다 (totalPeople...?)

wooody92 · 2022-05-25T10:05:15Z

BE/src/main/java/yanse/airbnb/domain/main/MainImage.java

+@AllArgsConstructor
+@NoArgsConstructor(access = AccessLevel.PROTECTED)
+@Entity
+public class MainImage {


Image 관련 엔티티가 세개정도 있는 것 같은데요. 하나의 엔티티에서 관리하기는 적합하지 않은 데이터들인가요??

생각해보니 하나의 엔티티에서 관리가 가능하여 해당 부분 수정할 예정입니다!

wooody92 · 2022-05-25T10:11:11Z

BE/src/main/java/yanse/airbnb/domain/member/Member.java

+	@OneToOne(fetch = LAZY)
+	@JoinColumn(name = "wishlist_id")
+	private WishList wishList;


Member와 WishList는 1:1관계인건가요?

해당 부분도 초기 엔티티 설계시 잘못 생각된 부분으로 Member와 WishList는 1:N 관계로 수정할 계획입니다!

wooody92 · 2022-05-25T10:18:50Z

BE/src/main/java/yanse/airbnb/domain/member/Member.java

+	@Column(length = 100, nullable = false)
+	private String name;


사람이름에 100자는 너무 길지 않을까요??

복붙의 폐해입니다...해당 부분도 필드에 맞게 변경하도록 하겠습니다!

7 검색바 UI 구현

* feat: 이슈 템플릿 추가 * feat: PR 템플릿 추가 (#3) * [Chore] 프로젝트 초기 설정 (#4) * chore: 프로젝트 환경 설정 (#3) * chore: Eslint import plugin, 스토리북 설정 (#3) Co-authored-by: suuzin <suuzin@users.noreply.github.com> * [Feat] Header 레이아웃 구현 (#8) * design: ThemeProvider 설정 (#5) Co-authored-by: suuzin <suuzin@users.noreply.github.com> * feat: svg 설정, 아이콘 파일 추가 (#5) Co-authored-by: suuzin <suuzin@users.noreply.github.com> * feat: Icon 컴포넌트 생성 (#5) * feat: Navigation 생성 (#5) * feat: Header, Logo, MyPageButton 생성 (#5) * feat: 라우터 설치, 헤더 렌더링 (#5) Co-authored-by: suuzin <suuzin@users.noreply.github.com> Co-authored-by: dongkyu-park <81552729+dongkyu-park@users.noreply.github.com> Co-authored-by: suuzin <suuzin@users.noreply.github.com>

Contents - NSMutableAttributedString 이용 각 Text 값 다르게 setting - NSMutableAttributedString extension으로Setting함수 추가

Contents - MapCard 하트버튼 추가 Todo - 버튼 액션 추가

Contents - houseNameLabel,pricePerDayLabel 생성 - Layout설정 - 화면 크기에따라 imageView크기 수정

�[iOS] ViewController가 여행지 추천을 요청하면 Dummy 데이터를 받을 수 있음

[#2, #3, #4] feat: SPM 작업, TabBarController 에 NavigationController 연결 작업

[iOS] refactor: SearchVC vc 이동 수정

…oad-asset [Android] MVVM패턴 구현 및 Search, PlaceSearch 화면 구현

Entity - User - Accommodation - AccommodationInfo - AccommodationPrice - Review - Region Enum - AccommodationType (숙소 타입)

Entity - Accommodation (One) - AccommodationInfo (toOne) - AccommodationPrice (toOne) - Review (toMany)

application.yml - MySQL 연결을 위한 설정 파일 추가

* [Feat] 모달 구현 (#8) * docs: 템플릿 수정(리뷰반영) * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * feat: 모달 구현 * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * build: 빌드 수정 * build: webpack styled options 수정 * [feat] 캘린더 한 달 뷰 (#11) * feat: 현재 달의 calendar를 모달에 출력 * fix: calendar날짜가 마지막 일까지 출력되도록 수정 * refactor: 캘린더 뷰에 사용하는 상수를 파일로 분리 * [feat] 차트 레이아웃 시도 (svg, canvas) (#12) * Feat: svg 그래프 * Feat: canvas 레이아웃 - 선 그리는 로직 미완 * Chore: svgChart에서 plots 좌표값 관련 주석 작성 Co-authored-by: HongJungKim(이든) <58525009+HongJungKim-dev@users.noreply.github.com> Co-authored-by: khj <hongjung.kim.dev@gmail.com>

- @types/styled-components 설치 jindonyy/airbnb/#8

jindonyy/airbnb/#8

* [sally4405/#4] Fix: API 주소 수정으로 인한 테스트 실패 문제 해결 * [sally4405-#4] Feat: 위시리스트 삭제 기능 Mock API 구현 * [sally4405-#4] Feat: 위시리스트 저장 기능 Mock API 구현 * [sally4405-#4] Feat: 위시리스트 조회 기능 Mock API 구현 * [sally4405-#4] Feat: 예약 상세 조회, 예약 취소 기능 Mock API 구현 * [sally4405-#4] Feat: 예약 저장 기능 Mock API 구현 * [sally4405/#4] Refactor: 인기 여행지 조회 방식 변경 - Service 계층을 통해 반환하고 있던 인기 여행지 조회 API를 고정 데이터라고 판단하여 Controller에서 바로 반환하도록 변경 * [sally4405/#4] Feat: 특정 지역 숙소 요금 조회 개발 - 특정 지역의 모든 1박 요금과 1박 요금의 평균을 나타내는 API 추가 * [sally4405/#4] Feat: 숙소 상세 정보 조회 - controller를 통해 입력된 id를 통해 해당 숙소 상세 정보를 조회하는 API 추가 * [sally4405/#4] Feat: 숙소 예약 요금 상세 조회 - id와 체크인, 체크아웃 날짜, 총 인원 수를 통해 상세 예약 요금을 반환해주는 API 추가 * [sally4405/#4] Refactor: 숙소 예약 요금 상세 조회시 dto를 통해 입력 받도록 변경 - 체크인, 체크아웃 날짜와 총 인원 수를 dto에 담아서 받도록 변경 - check-in, check-out 자료형 String에서 LocalDate로 변경 * [sally4405/#4] Refactor: 통합테스트명 변경 * [sally4405/#4] Feat: 입력한 값에 맞는 모든 숙소를 조회 * [sally4405/#4] Feat: 통합테스트 추가 - Spring rest docs 생성을 위한 통합테스트 추가 * [sally4405/#4] Feat: LocalDate 형식 변경을 위한 어노테이션 추가 - API 스펙에 맞게끔 LocalDate 형식을 변경 * [sally4405/#8] Feat: 도메인 객체 개발 Co-authored-by: Louie-03 <dhdustnr0134@naver.com> * [sally4405/#8] Refactor: Mock API 수정 Co-authored-by: Louie-03 <dhdustnr0134@naver.com> * [sally4405/#8] Docs: Spring Rest Docs API 문서 추가 Co-authored-by: Louie-03 <dhdustnr0134@naver.com> Co-authored-by: seok <dhdustnr0134@naver.com>

* [Feat] 모달 구현 (#8) * docs: 템플릿 수정(리뷰반영) * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * feat: 모달 구현 * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * build: 빌드 수정 * build: webpack styled options 수정 * [feat] 캘린더 한 달 뷰 (#11) * feat: 현재 달의 calendar를 모달에 출력 * fix: calendar날짜가 마지막 일까지 출력되도록 수정 * refactor: 캘린더 뷰에 사용하는 상수를 파일로 분리 * [feat] 차트 레이아웃 시도 (svg, canvas) (#12) * Feat: svg 그래프 * Feat: canvas 레이아웃 - 선 그리는 로직 미완 * Chore: svgChart에서 plots 좌표값 관련 주석 작성 * build: react-refresh 추가 * [PR] 1주차 두번째 PR 리뷰 반영 (#14) * refactor: 관심사 분리를 고려하여 모달 리팩토링 응집성을 위하여 html에서 모달 태그 삭제, Portal 컴포넌트에서 해당 element 생성 * refactor: interface 네이밍 수정 컴포넌트 이름 + Props (ex.ComponentProps) * build: liveReload 기능 false처리 * [Feat] 캘린더 렌더링 로직 구현 (#17) * Feat: 캘린더 렌더링 * Chore: 캘린더 관련 주석 작성 * [Feat] 차트 그래프 기본 구현 (#18) * feat: 가격 범위 차트 그래프 렌더링 * refactor: 중복 스타일 mixin으로 분리 중복된 캔버스 컨테이너 스타일 분리 Co-authored-by: HongJungKim(이든) <58525009+HongJungKim-dev@users.noreply.github.com> Co-authored-by: khj <hongjung.kim.dev@gmail.com>

* docs: 템플릿 수정(리뷰반영) * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * feat: 모달 구현

jeremy0405/airbnb/#8

- 각 Row에 해당하는 View와 ViewModel 구현 - HeadCountItemViewModel을 통해 CountLabel에 출력될 값과 Button의 enable 여부 조절 - HeadCountViewModel에 Button enable 조건검사 로직 구현 - StackView가 너무 많이 중첩되던 로직 간소화 jeremy0405/airbnb/#8

Feat: 위시리스트 조회기능 구현

ese111 and others added 11 commits May 23, 2022 10:23

파트 폴더 생성

3bfe588

Merge pull request #1 from ese111/main

24d074f

파트 폴더 생성

init: 프로젝트 초기설정

4349697

closes_#2

feat: 엔티티 만들기

5593333

- 연관관계 기본 설정 완료 closes_#3

Merge pull request #4 from ese111/feature_#3

1cfb0ba

feat: 엔티티 만들기

chore: gitignore 추가

26dfb4c

closes_#5

Merge pull request #6 from ese111/feature_#5

0bc975d

chore: gitignore 추가

refactor: 엔티티 수정및 추가

8eaa5a7

- 연관관계 매핑 수정 - room에 이미지 컬럼추가 및 main 도메인 생성 closes_#8

Merge pull request #9 from ese111/feature_#8

6153e5d

refactor: 엔티티 수정및 추가

chore: 환경변수 설정

d0033df

Merge pull request #12 from ese111/feature_#11

9dec254

chore: 환경변수 설정

rkolx added the review-BE Improvements or additions to documentation label May 25, 2022

rkolx requested a review from wooody92 May 25, 2022 07:42

rkolx assigned KTH-96 and rkolx May 25, 2022

wooody92 approved these changes May 25, 2022

View reviewed changes

wooody92 reviewed May 25, 2022

View reviewed changes

wooody92 merged commit becbc9d into codesquad-members-2022:team-16 May 25, 2022

ink-0 pushed a commit that referenced this pull request May 25, 2022

Merge pull request #8 from bukim0329/7-검색바-ui-구현

ff95aff

7 검색바 UI 구현

leejohy-0223 pushed a commit that referenced this pull request May 26, 2022

[feature] 후기 및 별점 Label 설정 [#8]

eabf18a

Contents - NSMutableAttributedString 이용 각 Text 값 다르게 setting - NSMutableAttributedString extension으로Setting함수 추가

leejohy-0223 pushed a commit that referenced this pull request May 26, 2022

[feature] 하트버튼 추가 [#8]

d445e60

Contents - MapCard 하트버튼 추가 Todo - 버튼 액션 추가

leejohy-0223 pushed a commit that referenced this pull request May 26, 2022

[feature] MapCardCell Label 마무리 [#8]

3acb0e6

Contents - houseNameLabel,pricePerDayLabel 생성 - Layout설정 - 화면 크기에따라 imageView크기 수정

naneun pushed a commit that referenced this pull request May 26, 2022

Merge pull request #8 from naneun/eddy-recommendation#7

d86873c

�[iOS] ViewController가 여행지 추천을 요청하면 Dummy 데이터를 받을 수 있음

SangHwi-Back added a commit that referenced this pull request May 27, 2022

Merge pull request #8 from ak2j38/feature-TabBar

ee63376

[#2, #3, #4] feat: SPM 작업, TabBarController 에 NavigationController 연결 작업

wnsxor1993 pushed a commit that referenced this pull request May 27, 2022

Merge pull request #8 from wnsxor1993/refactor/SearchVC_searchBar

0100b4a

[iOS] refactor: SearchVC vc 이동 수정

jminie-o8o added a commit that referenced this pull request May 27, 2022

feat: #8 MVVM 패턴 틀 잡기 및 Hilt 적용 및 HeroImage 불러오기 구현

3225be0

jminie-o8o added a commit that referenced this pull request May 27, 2022

feat: #8 CloseTour불러오기 및 툴바 스크롤 설정 구현

c58fc7d

jminie-o8o added a commit that referenced this pull request May 27, 2022

Merge pull request #19 from nathan29849/android/feature/#8-set-MVVM-l…

a0f635b

…oad-asset [Android] MVVM패턴 구현 및 Search, PlaceSearch 화면 구현

junseokseo9306 added a commit that referenced this pull request May 27, 2022

feat: 힐트 okhttp 모듈 생성 (#8)

6e47303

junseokseo9306 added a commit that referenced this pull request May 27, 2022

feat: DTO 및 모델 클래스 생성 (#8)

1abda3f

junseokseo9306 added a commit that referenced this pull request May 27, 2022

feat: homeContentApi 모듈 생성 (#8)

0462bc7

junseokseo9306 added a commit that referenced this pull request May 27, 2022

feat: DataSource 생성 (#8)

00cf5d9

junseokseo9306 added a commit that referenced this pull request May 27, 2022

feat: repository 파일 생성 (#8)

436c11d

junseokseo9306 added a commit that referenced this pull request May 27, 2022

feat: kotlin serialization을 사용해서 viewModel에 flow로 데이터 전달 (#8)

b2649bd

yeonnseok pushed a commit that referenced this pull request May 27, 2022

feat: Entity 객체 및 EnumType 추가(#8)

6d4a34d

Entity - User - Accommodation - AccommodationInfo - AccommodationPrice - Review - Region Enum - AccommodationType (숙소 타입)

yeonnseok pushed a commit that referenced this pull request May 27, 2022

feat: Entity 객체의 단방향 매핑(#8)

9de15e7

Entity - Accommodation (One) - AccommodationInfo (toOne) - AccommodationPrice (toOne) - Review (toMany)

yeonnseok pushed a commit that referenced this pull request May 27, 2022

feat: yml 설정 파일 추가(#8)

a153b86

application.yml - MySQL 연결을 위한 설정 파일 추가

choichoigang pushed a commit that referenced this pull request May 30, 2022

Refactor: style 파일 tsx로 변경

ce3d037

- @types/styled-components 설치 jindonyy/airbnb/#8

choichoigang pushed a commit that referenced this pull request May 30, 2022

Refactor: 절대 경로 아닌 부분 수정

c6639ac

jindonyy/airbnb/#8

deprecated-hongbiii pushed a commit that referenced this pull request Jun 6, 2022

[Feat] 모달 구현 (#8)

7a5e831

* docs: 템플릿 수정(리뷰반영) * refactor: config폴더내의 파일을 루트로 이동(리뷰반영) * build: react-refresh 추가 * feat: 모달 구현

ITzombietux pushed a commit that referenced this pull request Jun 8, 2022

feat: 인원 수 설정 뷰 레이아웃 및 버튼 기능 구현

ee24600

jeremy0405/airbnb/#8

Dae-Hwa pushed a commit that referenced this pull request Jun 12, 2022

Merge pull request #8 from nak253/feature-GET-wishList

ee8fbe4

Feat: 위시리스트 조회기능 구현

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Team-16][BE][Yanse] AirBnB 프로젝트 1주차 PR#1 #8

[Team-16][BE][Yanse] AirBnB 프로젝트 1주차 PR#1 #8

rkolx commented May 25, 2022

wooody92 left a comment

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

wooody92 May 25, 2022

rkolx May 26, 2022

		@AllArgsConstructor
		@NoArgsConstructor(access = AccessLevel.PROTECTED)

	package yanse.airbnb.domain.hotel;
	package yanse.airbnb.domain.room;

[Team-16][BE][Yanse] AirBnB 프로젝트 1주차 PR#1 #8

[Team-16][BE][Yanse] AirBnB 프로젝트 1주차 PR#1 #8

Conversation

rkolx commented May 25, 2022

초기 ERD

진행완료

wooody92 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

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

Choose a reason for hiding this comment