[team-21] 데이먼 첫 번째 PR 드립니다. #44

dukcode · 2022-05-25T08:34:48Z

안녕하세요 Dan!

이번 PR에는 많은 내용 없이 요구사항 분석, E-R Diagram 설계, 및 Entity 구현만을 진행했습니다.

다음 PR에는 더 발전된 프로젝트로 올리겠습니다.

요구사항 분석 및 , E-R Diagram Link

앞으로 3주간 잘 부탁드립니다.!

- Room, Booking, User, Wish, Review, Image, Amenity 구현 - H2 작동을 위한 application.yml 수정

Hyune-c

수고하셨습니다.

jpa 에 관련된 테스트를 많이 해보시면 좋을 것 같습니다.

Hyune-c · 2022-05-25T09:15:36Z

BE/src/main/java/team21/airbnb/domain/Booking.java

+public class Booking {
+
+    @Id
+    @GeneratedValue


@GeneratedValue의 strategy를 명시하는 것과 하지 않는 것에는 무슨 차이가 있을까요?

@GenerateValue의 strategy의 기본 값은 AUTO(DB의 기본생성전략을 따름)인데 h2의 기본 생성전략은 SEQUENCE, MySql의 기본 생성전략은 IDENTITY이므로 결국 배포시의 환경과 같이 맞추기 위해 IDENTITY로 명시적으로 적어주는게 맞을까요?

둘의 차이점에 대해 찾아봤는데 성능에 대한 이야기는 딱히 없어서요..

MySql의 기본 생성전략은 IDENTITY라는 것을 누가 정하고 있는지 알고 계신가요?

Hyune-c · 2022-05-25T09:15:57Z

BE/src/main/java/team21/airbnb/domain/Booking.java

+    private Integer numOfInfants;
+
+    @Enumerated(EnumType.STRING)
+    private BookingStatus status;


Hyune-c · 2022-05-25T09:19:43Z

BE/src/main/java/team21/airbnb/domain/Image.java

+
+@Entity
+@Getter
+@RequiredArgsConstructor(access = AccessLevel.PROTECTED)


Entity에서 @RequiredArgsConstructor 사용은 처음보는거 같아요. 혹시 어떤 의도인가요?
그리고 @NoArgsConstructor는 의도적으로 제거하신 건가요?

헉 @NoArgsConstructor를 사용한 것인데 오타로 RequiredArgsConstructor가 들어갔습니다... 수정해서 커밋올리겠습니다.

Hyune-c · 2022-05-25T09:19:54Z

BE/src/main/java/team21/airbnb/domain/Location.java

+import javax.persistence.Embeddable;
+import lombok.Getter;
+
+@Embeddable


- 기존 AllArgsConstructor로 잘못 설정된 annotation을 NoArgsConstructor로 수정

* feat: Room의 int field Wrapper 클래스로 변경, Builder 추가 * chore: DB 방언 SQL로 변경 * feat: Entity auto-increment 적용 * feat: RoomRepository 구현 * test: RoomRepositoryTest 구현 * feat: RoomChargeDistributionResponse DTO 구현 * feat: RoomService getRoomChargeDistribution() 메서드 구현 * test: RoomService test 구현 * feat: Entity 생성자 parameter wrapper 클래스로 변경 * feat: responseDTO field final 제거 및 기본생성자 추가 * feat: HomeController 구현 * test: HomeController 테스트 구현

Hyune-c

리뷰 후 추가로 올라오는 커밋은 해당 PR에 있던 리뷰 내용을 보완하는 것을 목표로 해주세요.
지금처럼 초기 구현 외의 것을 또 올려주시면 PR의 덩어리가 점점 커집니다.

대화 거리가 많지만 본 PR의 범위를 벗어나는 것으로 간주하여 이번 PR은 approve 합니다.

Hyune-c · 2022-05-26T01:38:45Z

BE/src/main/java/team21/airbnb/controller/HomeController.java

+
+    public HomeController(RoomService roomService) {
+        this.roomService = roomService;
+    }


nit
lombok을 활용할 수 있을 것 같아요.

@requiredargsconstructor 를 활용하겠습니다

Hyune-c · 2022-05-26T01:40:06Z

BE/src/main/java/team21/airbnb/domain/Room.java

+        this.weeklyDiscountPercent = weeklyDiscountPercent;
+        this.location = location;
+        this.host = host;
+    }


builder 패턴의 장단점을 고민하시고 다음 리뷰에 생각을 써주시면 좋을 것 같아요.

충분한 공부 이후 생각을 작성해보도록 하겠습니다

Hyune-c · 2022-05-26T01:44:52Z

BE/src/main/java/team21/airbnb/service/RoomService.java

+@Transactional(readOnly = true)
+public class RoomService {
+
+    public static final double DIGIT = 10.0;


저는 이게 무슨 변수인지 잘 모르겠어요.
네이밍을 더 고려하는 방법도 있겠지만, 적절한 주석도 좋은 코딩의 일부라고 생각합니다.

DISTRIBUTION_RANGE 가 더 적절한 이름 같습니다.!

Hyune-c · 2022-05-26T01:48:15Z

BE/src/main/java/team21/airbnb/service/RoomService.java

+        }
+
+        return new RoomChargeDistributionResponse(minRoomCharge, maxRoomCharge, graph);
+    }


이 메서드는 무슨 일을 하는건가요?
네이밍이 직관적이지 않고, 방의 전체 목록을 일단 조회하고 시작하는 것 부터가 일반적이진 않은 느낌이라고 생각해서 잘 이해가 안됩니다.

해당 메서드는

요금 버튼을 눌렀을 때 위 처럼 그래프를 구성할 수 있게
Room 가격에 따른 Room 갯수를 DIGIT 단위로 distribution하여 전달하는 메서드입니다.

Hyune-c · 2022-05-26T01:49:33Z

BE/src/test/java/team21/airbnb/controller/HomeControllerTest.java

+
+        // when
+        RoomChargeDistributionResponse roomChargeDistribution
+                = restTemplate.getForObject(url, RoomChargeDistributionResponse.class);


Spring에서는 resttemplate보다 좋은 웹 레이어 테스트 방법을 지원해줄 것 같아요.
조금 더 찾아보시면 좋을 것 같습니다.

좋은 방법을 찾아보겠습니다.

Hyune-c · 2022-05-26T01:50:28Z

BE/src/test/java/team21/airbnb/controller/HomeControllerTest.java

+
+    private int ceil(int num) {
+        return (int) (Math.ceil(num / RoomService.DIGIT) * RoomService.DIGIT);
+    }


이 메서드의 설계에 대해 데이먼의 생각을 알려주세요.
저는 개선이 필요하다고 생각합니다.

Room 에 ceilRoomCharge() 메서드를 만들어 책임을 넘기는 게 더 적절하다고 생각하는데 Dan이 의도한 사항이 맞을까요?

Hyune-c · 2022-05-26T01:50:56Z

BE/src/test/java/team21/airbnb/repository/RoomRepositoryTest.java

+        // then
+        Room findRoom = rooms.get(0);
+        assertThat(findRoom.getName()).isEqualTo(name);
+        assertThat(findRoom.getRoomCharge()).isEqualTo(roomCharge);


[Android] 안드로이드 폴더 생성

* Feature/3 (#26) * rename: gnb 폴더 생성하여 관련 컴포넌트 이동 - styled -> customStyled 로 변경 적용 * refactor: gnb 컴포넌트 css 수정 * feat: main banner UI 구현 * refactor: gnb 컴포넌트 hover 및 cursor pointer 적용 * Feature/2 (#27) * chore: mui 버전 통일 * feat: 글로벌 스타일 추가 * feat: SearchBar 컴포넌트 style 작성 및 이벤트 추가 * Feature/alias (#28) * chore: alias 경로 설정을 위한 react-app-rewired 설치 및 설정파일 생성 * refactor: 절대경로 반영 (index, app, gnb, main-banner) * fix: search input width 고정 * Feature/4 (#29) * feat: calender page component 구현 * feat: 달력 월~일요일 부분 컴포넌트 구현 * feat: 해당 연-월에 맞는 날짜 영역 컴포넌트 구현 * feat: 이전, 다음 버튼 컴포넌트 및 클릭 기능 구현 * feat: calender 컴포넌트 구현 - 보여줄 달력 개수를 정하는 page 옵션 - curData 기반으로 page 만큼의 달력 렌더링 * chore: 브라우저 확인용 App 에 calender 반영 * feat: calender 관련 상태 context 사용하여 분리 및 provider 컴포넌트 생성 - 불필요한 주석 제거 * refactor: 컴포넌트 이름 변경, DatesOfMonth 에서 DateBox 컴포넌트 분리 * feat: DateBox 컴포넌트 구현 - 클릭 시 checkIn, checkOut 상태 변경 - checkIn, checkOut 날짜에 따라 동적 css 적용 * chore: 오타 수정, app.js 에 provider 적용 * feat: header 컴포넌트 생성 - gnb, search-bar, calender를 header 하위 컴포넌트로 수정 * Feature/30 (#32) * refactor: 캘린더 체크인/체크아웃 모드와 date box 체크 위치 상태를 나타내는 문자열을 상수화 * refactor: 컴포넌트 별 데이터 가공 유틸 함수를 컴포넌트 밖으로 분리 * refactor: Date box 내부에 있던 날짜 비교용 checkInTime, checkOutTime 을 콘텍스트 provider 로 이동 - checkInDate, checkOutDate 변수명 변경 -> checkInInfo, checkOutInfo - checkIn, checkOut, current 변수명 변경 -> checkInTime, checkOutTime, currentTime * refactor: 캘린더 month 빈배열 생성 시 0 -> Null 로 변경 * Feature/31 (#33) * refactor: 코드 리뷰 반영 - visibility 속성 display로 변경 - bool 타입 결과값 함수 중복 검사 제거 - SearchMenu 컴포넌트 div 태그로 변경 * feat & refactor: SearchBar 컴포넌트 컨텍스트 추가 - props로 내려주던 상태 context로 관리 하도록 함 * feat : 검색바 반응형 컴포넌트로 수정 * refactor: isFocus를 상태 관리에서 제외 - currentInput의 상태를 통해 얻을 수 있도록 함 * feat&refactor: Header 컴포넌트 반응형으로 수정 - GNB, MainBanner, SearchBar 모두 적용 * feat: 체크인, 체크아웃 인풋 영역 클릭 시 캘린더 모달 팝업 기능 추가 * feat: Header 컴포넌트 fixed 속성 추가 * feat: 모달창 클릭시 검색바 blur가 적용되지 않게 함 * Feature/34 (#36) * feat: 체크인-체크아웃 사이 날짜 1일과 마지막일 전후 음영 반영 - DatesOfMonth 컴포넌트 last date 월이 안맞는 오류 수정 : getDate() 는 현재달 인덱스에 date 0을 주면 이전달 마지막날 반환 * feat: 현재 날짜 이전 날짜 선택 불가 및 회색 표시 * feat: 검색바 체크인, 체크아웃 날짜 출력 기능 추가 (#38) * feat: 검색바 체크인, 체크아웃 날짜 출력 기능 추가 - 캘린더에서 선택한 날짜를 출력하도록 함 * Feature/20 (#40) * refactor: ResetButton 동적으로 추가하도록 수정 - input요소의 value 유무에 따라 추가, 삭제 * feat: ResetButton 클릭 이벤트 추가 - 버튼 클릭시 input value 초기화 - 검색바가 포커스되어 있을 때만 버튼이 노출되도록 수정 * feat: 검색바 캘린더 체크인 체크아웃 상태 연동 (#41) * fix: 포커스 이벤트 에러 수정 - ResetButton 포커스 시에만 노출되도록 수정 - 검색바 포커스 시 세로로 늘어나는 오류 수정 * fix: 캘린더 날짜 선택 오류 수정 - 체크아웃을 먼저 선택 후 체크인 선택 시 체크아웃보다 뒷날이어도 선택되는 오류 수정 - DateBox 컴포넌트 내 handle click 로직 수정 * feat: 캘린더 모달 선택 시 보여지는 달 업데이트 - 체크인 날짜 있는 경우 체크인 달로 시작 - 없는 경우 현재 달로 시작 * fix: 캘린더 토요일과 날짜 정렬 * Feature/15 (#42) * refactor: 모달 배경 관련 스타일 별도 컴포넌트로 분리 * rename: custom-styled-component 폴더 utils 하위로 이동 - Header.js => Header.jsx 로 변경 * rename : calender 폴더 modal 하위 폴더로 이동 - 절대 경로 설정 및 적용 * refactor: ModalContainer 스타일 theme 사용 하도록 수정 * feat: Personnel 모달 컴포넌트 추가 * Feature/7 (#43) * rename: context 폴더 분리 - provider 모두 context 폴더 하위로 이동 * feat: 인원 설정 모달 컨트롤러 버튼 컴포넌트 분리 * feat: 검색바 인원 상태 관리를 위한 관련 상수 추가 * feat: 검색바 인원 설정 모달 인원 추가, 인원 삭제 기능 추가 * feat: 검색바에 선택된 인원 출력 기능 추가 (#44) * feat: 검색바에 선택된 인원 출력 기능 추가 * Feature/11 (#45) * feat: 성인 없이 어린이, 유아 인원 추가 방지 기능 추가 * fix: 성인 인원을 줄일 수 없는 상황에 버튼 비활성화 색상 스타일 적용 * refactor: ResetButton 컴포넌트 onClick 핸들러 props로 받도록 수정 * feat: 인원 설정 초기화 기능 추가 * refactor&style: 중복코드 삭제, 줄띄움 * fix: 검색바 초기화 버튼 유무따라 크기가 변경되는 부분 수정 - 검색 버튼 아이콘 컬러 변경 * fix: 검색바 사이즈 줄면 초기화 버튼이 검색 버튼에 가리는 오류 수정 * Feature/46 (#47) * refactor: 중복 코드 함수화, StyledExpandBackground 분기문 - info 를 받아 time 으로 바꿔주는 함수 - 체크인-체크아웃 사이에 있고, 1일이거나 마지막날일 경우에만 StyledExpandBackround 컴포넌트 생성 * feat: calender provider 에 prev date, next date 추가 - 불필요한 useEffect 코드 제거 * feat: calender 컴포넌트에서 page 생성 부분을 calender carousel 컴포넌트로 이동 - calender position 상태 생성 * chore: calender page width 수정(336 -> 350px) * feat: 이전, 다음달 이동 버튼 로직 수정 - curDate 를 직접 바꾸는 방식에서 calender position 만 바꿔주는 방식 - calender position 에 따라 carousel에서 translateX 후 curDate 변경 * feat: Calender Carousel 컴포넌트 구현 - prevDate, curDate, nextDate 기준으로 각각 Calender page wrapper 로 묶음 - calender display 영역 내에서 moveArea 를 이동 * fix: 검색바 모달 관련 인풋창 아래에 나타나도록 위치 수정 Co-authored-by: bangdler <90082464+bangdler@users.noreply.github.com> Co-authored-by: bangdler <zbthz90@gmail.com>

Sally review1

[iOS] FindAccomodation 도메인 리팩토링 진행

[BE] - PR 2-1 Review 반영

…_by_fee feat: [44] 금액 구간에 따른 숙소 조회 기능 구현

* Feature/3 (#26) * rename: gnb 폴더 생성하여 관련 컴포넌트 이동 - styled -> customStyled 로 변경 적용 * refactor: gnb 컴포넌트 css 수정 * feat: main banner UI 구현 * refactor: gnb 컴포넌트 hover 및 cursor pointer 적용 * Feature/2 (#27) * chore: mui 버전 통일 * feat: 글로벌 스타일 추가 * feat: SearchBar 컴포넌트 style 작성 및 이벤트 추가 * Feature/alias (#28) * chore: alias 경로 설정을 위한 react-app-rewired 설치 및 설정파일 생성 * refactor: 절대경로 반영 (index, app, gnb, main-banner) * fix: search input width 고정 * Feature/4 (#29) * feat: calender page component 구현 * feat: 달력 월~일요일 부분 컴포넌트 구현 * feat: 해당 연-월에 맞는 날짜 영역 컴포넌트 구현 * feat: 이전, 다음 버튼 컴포넌트 및 클릭 기능 구현 * feat: calender 컴포넌트 구현 - 보여줄 달력 개수를 정하는 page 옵션 - curData 기반으로 page 만큼의 달력 렌더링 * chore: 브라우저 확인용 App 에 calender 반영 * feat: calender 관련 상태 context 사용하여 분리 및 provider 컴포넌트 생성 - 불필요한 주석 제거 * refactor: 컴포넌트 이름 변경, DatesOfMonth 에서 DateBox 컴포넌트 분리 * feat: DateBox 컴포넌트 구현 - 클릭 시 checkIn, checkOut 상태 변경 - checkIn, checkOut 날짜에 따라 동적 css 적용 * chore: 오타 수정, app.js 에 provider 적용 * feat: header 컴포넌트 생성 - gnb, search-bar, calender를 header 하위 컴포넌트로 수정 * Feature/30 (#32) * refactor: 캘린더 체크인/체크아웃 모드와 date box 체크 위치 상태를 나타내는 문자열을 상수화 * refactor: 컴포넌트 별 데이터 가공 유틸 함수를 컴포넌트 밖으로 분리 * refactor: Date box 내부에 있던 날짜 비교용 checkInTime, checkOutTime 을 콘텍스트 provider 로 이동 - checkInDate, checkOutDate 변수명 변경 -> checkInInfo, checkOutInfo - checkIn, checkOut, current 변수명 변경 -> checkInTime, checkOutTime, currentTime * refactor: 캘린더 month 빈배열 생성 시 0 -> Null 로 변경 * Feature/31 (#33) * refactor: 코드 리뷰 반영 - visibility 속성 display로 변경 - bool 타입 결과값 함수 중복 검사 제거 - SearchMenu 컴포넌트 div 태그로 변경 * feat & refactor: SearchBar 컴포넌트 컨텍스트 추가 - props로 내려주던 상태 context로 관리 하도록 함 * feat : 검색바 반응형 컴포넌트로 수정 * refactor: isFocus를 상태 관리에서 제외 - currentInput의 상태를 통해 얻을 수 있도록 함 * feat&refactor: Header 컴포넌트 반응형으로 수정 - GNB, MainBanner, SearchBar 모두 적용 * feat: 체크인, 체크아웃 인풋 영역 클릭 시 캘린더 모달 팝업 기능 추가 * feat: Header 컴포넌트 fixed 속성 추가 * feat: 모달창 클릭시 검색바 blur가 적용되지 않게 함 * Feature/34 (#36) * feat: 체크인-체크아웃 사이 날짜 1일과 마지막일 전후 음영 반영 - DatesOfMonth 컴포넌트 last date 월이 안맞는 오류 수정 : getDate() 는 현재달 인덱스에 date 0을 주면 이전달 마지막날 반환 * feat: 현재 날짜 이전 날짜 선택 불가 및 회색 표시 * feat: 검색바 체크인, 체크아웃 날짜 출력 기능 추가 (#38) * feat: 검색바 체크인, 체크아웃 날짜 출력 기능 추가 - 캘린더에서 선택한 날짜를 출력하도록 함 * Feature/20 (#40) * refactor: ResetButton 동적으로 추가하도록 수정 - input요소의 value 유무에 따라 추가, 삭제 * feat: ResetButton 클릭 이벤트 추가 - 버튼 클릭시 input value 초기화 - 검색바가 포커스되어 있을 때만 버튼이 노출되도록 수정 * feat: 검색바 캘린더 체크인 체크아웃 상태 연동 (#41) * fix: 포커스 이벤트 에러 수정 - ResetButton 포커스 시에만 노출되도록 수정 - 검색바 포커스 시 세로로 늘어나는 오류 수정 * fix: 캘린더 날짜 선택 오류 수정 - 체크아웃을 먼저 선택 후 체크인 선택 시 체크아웃보다 뒷날이어도 선택되는 오류 수정 - DateBox 컴포넌트 내 handle click 로직 수정 * feat: 캘린더 모달 선택 시 보여지는 달 업데이트 - 체크인 날짜 있는 경우 체크인 달로 시작 - 없는 경우 현재 달로 시작 * fix: 캘린더 토요일과 날짜 정렬 * Feature/15 (#42) * refactor: 모달 배경 관련 스타일 별도 컴포넌트로 분리 * rename: custom-styled-component 폴더 utils 하위로 이동 - Header.js => Header.jsx 로 변경 * rename : calender 폴더 modal 하위 폴더로 이동 - 절대 경로 설정 및 적용 * refactor: ModalContainer 스타일 theme 사용 하도록 수정 * feat: Personnel 모달 컴포넌트 추가 * Feature/7 (#43) * rename: context 폴더 분리 - provider 모두 context 폴더 하위로 이동 * feat: 인원 설정 모달 컨트롤러 버튼 컴포넌트 분리 * feat: 검색바 인원 상태 관리를 위한 관련 상수 추가 * feat: 검색바 인원 설정 모달 인원 추가, 인원 삭제 기능 추가 * feat: 검색바에 선택된 인원 출력 기능 추가 (#44) * feat: 검색바에 선택된 인원 출력 기능 추가 * Feature/11 (#45) * feat: 성인 없이 어린이, 유아 인원 추가 방지 기능 추가 * fix: 성인 인원을 줄일 수 없는 상황에 버튼 비활성화 색상 스타일 적용 * refactor: ResetButton 컴포넌트 onClick 핸들러 props로 받도록 수정 * feat: 인원 설정 초기화 기능 추가 * refactor&style: 중복코드 삭제, 줄띄움 * fix: 검색바 초기화 버튼 유무따라 크기가 변경되는 부분 수정 - 검색 버튼 아이콘 컬러 변경 * fix: 검색바 사이즈 줄면 초기화 버튼이 검색 버튼에 가리는 오류 수정 * Feature/46 (#47) * refactor: 중복 코드 함수화, StyledExpandBackground 분기문 - info 를 받아 time 으로 바꿔주는 함수 - 체크인-체크아웃 사이에 있고, 1일이거나 마지막날일 경우에만 StyledExpandBackround 컴포넌트 생성 * feat: calender provider 에 prev date, next date 추가 - 불필요한 useEffect 코드 제거 * feat: calender 컴포넌트에서 page 생성 부분을 calender carousel 컴포넌트로 이동 - calender position 상태 생성 * chore: calender page width 수정(336 -> 350px) * feat: 이전, 다음달 이동 버튼 로직 수정 - curDate 를 직접 바꾸는 방식에서 calender position 만 바꿔주는 방식 - calender position 에 따라 carousel에서 translateX 후 curDate 변경 * feat: Calender Carousel 컴포넌트 구현 - prevDate, curDate, nextDate 기준으로 각각 Calender page wrapper 로 묶음 - calender display 영역 내에서 moveArea 를 이동 * fix: 검색바 모달 관련 인풋창 아래에 나타나도록 위치 수정 * Feature/15 1 (#49) * feat: Price 컴포넌트 생성 - Title 컴포넌트 재사용을 위해 분리 * refactor: 검색바 블러 이벤트 대신 dim레이어 클릭 시 모달이 사라지도록 수정 - e.preventDefault 때문에 range input의 thumb가 움직이지 않아 수정 * feat: 요금 설정 모달 요금별 데이터 그래프 추가 - 임시 데이터 추가 * feat: 요금 범위 설정 슬라이더 추가 * chore: chart.js 설정 추가 * fix: MainBanner 이미지 연결 오류 수정 * feat: 요금 모달 세부 텍스트 출력 * feat: 검색바에 요금 상태 출력 - 초기화 버튼 이벤트 핸들러 추가 * fix: merge 후 모달창이 관련 인풋창 아래에 위치하지 않는 오류 수정 * refactor: 인원 모달 컨트롤 버튼 하나의 컴포넌트로 합침 - type으로 add, remove 구분하도록 props 추가 * refactor: calender Prev Button, Next Button 을 통합한 Calender Button 구현 (#50) Co-authored-by: moon-yerim <75062526+moonyerim2@users.noreply.github.com> * refactor: 모달창 클릭 시 닫히지 않도록 하는 기능 수정 - dim레이어 삭제 - Node.contains를 사용한 방법으로 수정 Co-authored-by: bangdler <90082464+bangdler@users.noreply.github.com> Co-authored-by: bangdler <zbthz90@gmail.com>

dukcode added 2 commits May 24, 2022 16:59

init project

3c8d528

feat(#7): Entity 구현

86790d7

- Room, Booking, User, Wish, Review, Image, Amenity 구현 - H2 작동을 위한 application.yml 수정

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

dukcode requested a review from Hyune-c May 25, 2022 08:34

dukcode self-assigned this May 25, 2022

Hyune-c requested changes May 25, 2022

View reviewed changes

fix: Image, Wish 엔티티 Constructor(All -> No) 수정함

f95dbec

- 기존 AllArgsConstructor로 잘못 설정된 annotation을 NoArgsConstructor로 수정

dukcode requested a review from Hyune-c May 25, 2022 21:30

Hyune-c approved these changes May 26, 2022

View reviewed changes

dukcode merged commit 1841854 into codesquad-members-2022:team-21 May 26, 2022

dukcode mentioned this pull request May 27, 2022

[team-21] 데이먼 1주차 2번째 PR 요청 #116

Merged

sabgilhun pushed a commit that referenced this pull request Jun 3, 2022

Merge pull request #44 from hwicode/feature/price-range

eb2c31a

[Android] 안드로이드 폴더 생성

tmdgusya pushed a commit that referenced this pull request Jun 4, 2022

Merge pull request #44 from sally-ksh/sally-review1

744b4a8

Sally review1

wnsxor1993 added a commit that referenced this pull request Jun 6, 2022

Merge pull request #44 from wnsxor1993/refactor/FindAccomodation

241916e

[iOS] FindAccomodation 도메인 리팩토링 진행

yeonnseok pushed a commit that referenced this pull request Jun 6, 2022

Merge pull request #44 from jeremy0405/BE-feature/PR_REVIEW_2_1

80e1585

[BE] - PR 2-1 Review 반영

wooody92 pushed a commit that referenced this pull request Jun 9, 2022

Merge pull request #66 from nathan29849/be/feature/#44-accommodations…

0c0bde8

…_by_fee feat: [44] 금액 구간에 따른 숙소 조회 기능 구현

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[team-21] 데이먼 첫 번째 PR 드립니다. #44

[team-21] 데이먼 첫 번째 PR 드립니다. #44

dukcode commented May 25, 2022

Hyune-c left a comment

Hyune-c May 25, 2022

dukcode May 25, 2022

Hyune-c May 26, 2022 •

edited

Loading

Hyune-c May 25, 2022

Hyune-c May 25, 2022

dukcode May 25, 2022

Hyune-c May 25, 2022

Hyune-c left a comment

Hyune-c May 26, 2022

dukcode May 26, 2022

Hyune-c May 26, 2022

dukcode May 26, 2022

Hyune-c May 26, 2022

dukcode May 26, 2022

Hyune-c May 26, 2022

dukcode May 26, 2022

Hyune-c May 26, 2022

dukcode May 26, 2022

Hyune-c May 26, 2022

dukcode May 26, 2022

Hyune-c May 26, 2022

[team-21] 데이먼 첫 번째 PR 드립니다. #44

[team-21] 데이먼 첫 번째 PR 드립니다. #44

Conversation

dukcode commented May 25, 2022

Hyune-c left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Hyune-c May 26, 2022 • 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

Hyune-c 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

Hyune-c May 26, 2022 •

edited

Loading