[TEAM-25] Airbnb 2주차 1번째 PR Nick + Benny #169
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Initialize project using Create React App * feat: 개발 환경 셋팅 create-react-app(typescript) styled-components(typescript) styled-reset eslint prettier material ui * refactor: 개발 환경 셋팅 수정 * feat: 디렉터리, 폰트, styled-components 환경 구성 global.js 추가 theme.js 추가 google fonts 추가 * feat: 절대경로 설정 추가 * feat: router 설정 * feate: components에 컴포넌트별 디렉터리 생성 * feat: Hero 컴포넌트 마크업, 스타일 추가 * feat: constant 디렉터리, constant.js 파일 추가 * feat: utils.js 파일 추가 * design: utils.js WrapperMarginCenter 수정 * feat: 중첩 router 사용해 공통 header 추가 * design: Hero, Home 컴포넌트 스타일 수정 * feat: GNB 컴포넌트 마크업, 스타일 추가 * refactor: 코드리뷰 피드백 수정 * refactor: Hero, GNB 마크업 스타일 수정 * feat: header 마크업, 스타일 추가 * remove: js 파일 제거 * refactor: header 마크업 스타일 리팩토링 * feat: search 마크업, 스타일 추가 * style: favicon 추가 * feat: modal portal 기능 추가 * refactor: type들 파일로 따로 분류 * feat: modal backdrop show, hide 기능 추가 * feat: 캘린더 모달 내 캘린더 버튼 추가 * feat: Calendar 년, 월에 따른 일수 생성 기능 추가 * feat: 캐러셀 움직이는 기능 추가 * feat: 캘린더 새로운 년, 월 추가하는 기능 추가 * feat: 캘린더 마지막 요소일때만 추가하는 기능 오른쪽 버튼을 누를때 보여지는 요소가 캘린더의 마지막 아이템일경우에만 캘 린더를 추가하는 기능 추가 * style: 불필요한 주석 제거 * style: 캘린더 '일'에 마우스 호버시 스타일 추가 color, background-color 변경 * refactor: 중복되는 함수, type 변경 * feat: 과거 일자 스타일, 클릭 방지 기능 추가 * fix: 캘린더에서 오늘 날짜를 비활성시키는 버그 픽스 * feat: DateCell 컴포넌트 추가 * refactor: 코드리뷰 피드백 수정 * style: CalendarModal 변수명 리네임, 상수 추가 * refactor: CalendarModal reducer 부분 리팩토링 * rename: 파일들 FE 폴더로 이동 * feat: 라우터를 이용시 공통으로 보이는 배경 수정 * feat: 캘린더 요소 클릭시 선택되는 기능 구현 * fix: 체크인 관련 버그 수정 체크인을 누른 후 체크인보다 작은 날 누를시 생기는 버그 수정 * feat: 캘린더 체크인, 체크아웃 기능 구현 완료
ghost
added
the
ghost
requested a review
from 
somedaycode
approved these changes
May 31, 2022
Comment on lines
+12
to
+18
| "@testing-library/jest-dom": "^5.16.4", | ||
| "@testing-library/react": "^13.2.0", | ||
| "@testing-library/user-event": "^13.5.0", | ||
| "@types/jest": "^27.5.1", | ||
| "@types/node": "^16.11.36", | ||
| "@types/react": "^18.0.9", | ||
| "@types/react-dom": "^18.0.5", |
Comment on lines
+75
to
+90
| const datesCells = dates.map(el => { | ||
| if (el.date === 0) | ||
| return <DateCell key={el.id} calendarClickCount={calendarClickCount} />; | ||
|
|
||
| const past = distinguishPast(el); | ||
|
|
||
| return ( | ||
| <DateCell | ||
| key={el.id} | ||
| dateInfo={el} | ||
| past={past} | ||
| calendarClickCount={calendarClickCount} | ||
| /> | ||
| ); | ||
| }); | ||
|
|
|
|
||
| function createTotalDateArr( | ||
| totalDate: number, | ||
| curMonthOneDateDay: number, |
There was a problem hiding this comment.
curMonthOneDateDay 어떤 걸 뜻하는지 잘 이해가 안되네요 ㅎㅎ
Comment on lines
+45
to
+49
| const curMonthOneDateDay: number = new Date( | ||
| dateInfo.year, | ||
| dateInfo.month - 1, | ||
| 1, | ||
| ).getDay(); |
There was a problem hiding this comment.
const curMonthOneDateDay: number = new Date(
dateInfo.year,
dateInfo.month - 1,
1,
).getDay();number 타입이 추론이 되고 있다면 따로 작성해주지 않아도 좋을 것 같아요 ㅎㅎ
| dateInfo, | ||
| ); | ||
|
|
||
| const week = DAY_TEXTS.map(el => <WeekCell key={el}>{el}</WeekCell>); |
There was a problem hiding this comment.
const week = DAY_TEXTS.map(day => <WeekCell key={day}>{day}</WeekCell>);el 보다는 day가 코드를 이해하는게 더 도움이 될 것 같아요
또 DAY_TEXTS 보다는 DAYS 로 작성하셔도 괜찮은 것 같기도 하네요
| } | ||
| }; | ||
|
|
||
| const handleClickDateCellContent = () => { |
There was a problem hiding this comment.
먼저 new Date를 사용한 부분들이 굉장히 많은데 어떻게 중복을 제거하면 좋을지 고민을 하는게 좋을 것 같아요
DRY에 대한 링크를 공유드립니다
https://johngrib.github.io/wiki/dry-principle/
또 조건에 따른 분기문이 굉장히 많은데 이런 부분들도 가독성에 대해 함께 고민해보면 좋을 것 같네요
There was a problem hiding this comment.
무엇보다 handleClickDateCellContent 함수가 꽤 크죠 ㅎㅎ
| onMouseOver={handleMouseOverDateCellContent} | ||
| > | ||
| <DateCellCircle past={past} dateInfo={dateInfo} rangeFlag={rangeFlag}> | ||
| {dateInfo?.date} |
There was a problem hiding this comment.
? 옵셔널체이닝을 자주쓰시는데 단순히 타입의 문제를 해결하기 위한 방법이 아니길 바랍니다 ㅎㅎ
https://www.typescriptlang.org/docs/handbook/2/narrowing.html#typeof-type-guards
| calendarClickCount: { current: number }; | ||
| } | ||
|
|
||
| export interface DateCellCircleProps extends DateCell { |
Comment on lines
+35
to
+48
| <SearchBar> | ||
| <SearchForm> | ||
| <CheckContext.Provider value={checkContext}> | ||
| <Check /> | ||
| </CheckContext.Provider> | ||
| <Stick /> | ||
| <Fare /> | ||
| <Stick /> | ||
| <Personnel /> | ||
| <SearchBtn type="submit"> | ||
| <img src="./assets/images/search.svg" alt="검색" /> | ||
| </SearchBtn> | ||
| </SearchForm> | ||
| </SearchBar> |
Comment on lines
+22
to
+32
| const checkContext: checkContextInterface = useMemo( | ||
| () => ({ | ||
| checkIn, | ||
| checkOut, | ||
| setCheckIn, | ||
| setCheckOut, | ||
| mouseOverCheckOut, | ||
| setMouseOverCheckOut, | ||
| }), | ||
| [checkIn, checkOut, mouseOverCheckOut], | ||
| ); |
There was a problem hiding this comment.
useCallback과 useMemo의 차이는 뭘까요?
https://reactjs.org/docs/hooks-reference.html#usecallback
https://reactjs.org/docs/hooks-reference.html#usememo
jinan159
added a commit
to jinan159/airbnb
that referenced
this pull request
Jun 8, 2022
* Initialize project using Create React App * feat: 개발 환경 셋팅 create-react-app(typescript) styled-components(typescript) styled-reset eslint prettier material ui * refactor: 개발 환경 셋팅 수정 * feat: 디렉터리, 폰트, styled-components 환경 구성 global.js 추가 theme.js 추가 google fonts 추가 * feat: 절대경로 설정 추가 * feat: router 설정 * feate: components에 컴포넌트별 디렉터리 생성 * feat: Hero 컴포넌트 마크업, 스타일 추가 * feat: constant 디렉터리, constant.js 파일 추가 * feat: utils.js 파일 추가 * design: utils.js WrapperMarginCenter 수정 * feat: 중첩 router 사용해 공통 header 추가 * design: Hero, Home 컴포넌트 스타일 수정 * feat: GNB 컴포넌트 마크업, 스타일 추가 * [TEAM-25] Airbnb 1주차 1번째 PR Nick + Benny (#45) * Initialize project using Create React App * feat: 개발 환경 셋팅 create-react-app(typescript) styled-components(typescript) styled-reset eslint prettier material ui * refactor: 개발 환경 셋팅 수정 * feat: 디렉터리, 폰트, styled-components 환경 구성 global.js 추가 theme.js 추가 google fonts 추가 * feat: 절대경로 설정 추가 * feat: router 설정 * feate: components에 컴포넌트별 디렉터리 생성 * feat: Hero 컴포넌트 마크업, 스타일 추가 * feat: constant 디렉터리, constant.js 파일 추가 * feat: utils.js 파일 추가 * design: utils.js WrapperMarginCenter 수정 * feat: 중첩 router 사용해 공통 header 추가 * design: Hero, Home 컴포넌트 스타일 수정 * feat: GNB 컴포넌트 마크업, 스타일 추가 Co-authored-by: Ubeom Kim <kimub1204@gmail.com> * Revert "[TEAM-25] Airbnb 1주차 1번째 PR Nick + Benny (#45)" This reverts commit e81f2a6. * [TEAM-25] Airbnb 1주차 2번째 PR Nick + Benny (codesquad-members-2022#96) * refactor: 코드리뷰 피드백 수정 * refactor: Hero, GNB 마크업 스타일 수정 * feat: header 마크업, 스타일 추가 * remove: js 파일 제거 * refactor: header 마크업 스타일 리팩토링 * feat: search 마크업, 스타일 추가 * style: favicon 추가 * feat: modal portal 기능 추가 Co-authored-by: Ubeom Kim <kimub1204@gmail.com> * [FE] 체크인, 체크아웃 기능 구현 완료 (#25) * Initialize project using Create React App * feat: 개발 환경 셋팅 create-react-app(typescript) styled-components(typescript) styled-reset eslint prettier material ui * refactor: 개발 환경 셋팅 수정 * feat: 디렉터리, 폰트, styled-components 환경 구성 global.js 추가 theme.js 추가 google fonts 추가 * feat: 절대경로 설정 추가 * feat: router 설정 * feate: components에 컴포넌트별 디렉터리 생성 * feat: Hero 컴포넌트 마크업, 스타일 추가 * feat: constant 디렉터리, constant.js 파일 추가 * feat: utils.js 파일 추가 * design: utils.js WrapperMarginCenter 수정 * feat: 중첩 router 사용해 공통 header 추가 * design: Hero, Home 컴포넌트 스타일 수정 * feat: GNB 컴포넌트 마크업, 스타일 추가 * refactor: 코드리뷰 피드백 수정 * refactor: Hero, GNB 마크업 스타일 수정 * feat: header 마크업, 스타일 추가 * remove: js 파일 제거 * refactor: header 마크업 스타일 리팩토링 * feat: search 마크업, 스타일 추가 * style: favicon 추가 * feat: modal portal 기능 추가 * refactor: type들 파일로 따로 분류 * feat: modal backdrop show, hide 기능 추가 * feat: 캘린더 모달 내 캘린더 버튼 추가 * feat: Calendar 년, 월에 따른 일수 생성 기능 추가 * feat: 캐러셀 움직이는 기능 추가 * feat: 캘린더 새로운 년, 월 추가하는 기능 추가 * feat: 캘린더 마지막 요소일때만 추가하는 기능 오른쪽 버튼을 누를때 보여지는 요소가 캘린더의 마지막 아이템일경우에만 캘 린더를 추가하는 기능 추가 * style: 불필요한 주석 제거 * style: 캘린더 '일'에 마우스 호버시 스타일 추가 color, background-color 변경 * refactor: 중복되는 함수, type 변경 * feat: 과거 일자 스타일, 클릭 방지 기능 추가 * fix: 캘린더에서 오늘 날짜를 비활성시키는 버그 픽스 * feat: DateCell 컴포넌트 추가 * refactor: 코드리뷰 피드백 수정 * style: CalendarModal 변수명 리네임, 상수 추가 * refactor: CalendarModal reducer 부분 리팩토링 * rename: 파일들 FE 폴더로 이동 * feat: 라우터를 이용시 공통으로 보이는 배경 수정 * feat: 캘린더 요소 클릭시 선택되는 기능 구현 * fix: 체크인 관련 버그 수정 체크인을 누른 후 체크인보다 작은 날 누를시 생기는 버그 수정 * feat: 캘린더 체크인, 체크아웃 기능 구현 완료 * [Team-25][BE][Jay] AWS 인프라 & CI/CD 환경 구축 및 숙소 통계 API 반영 PR (codesquad-members-2022#143) * Update issue templates * Update issue templates * Update issue templates * Update issue templates * Initial commit * build: Github action 백엔 개발 CI 에 working-directory 추가 * build: Github action 백엔드 운영 CI/CD 에 working-directory 추가 * feat: 전역 예외 클래스 추가 비즈니스 예외 클래스들이 HttpMethod 를 반환하도록 하여, 각 예외에 맞는 HTTP 상태코드를 반환할 수 있도록 함 * feat: 전역 예외 로직 추가 프로젝트 전역에서 발생하는 예외를 처리하는 로직 추가 - 비즈니스 예외 발생시에는 예외 클래스에 있는 HTTP 상태코드 반환 - 나머지 분류되지 않은 예외는 INTERNAL_SERVER_ERROR 로 반환 * feat: DB 연결 설정 추가 * feat: airbnb 테이블 정의(DDL) 추가 * feat: airbnb Entity 추가 * feat: 숙소(Accommodation) 에 가격 추가 * feat: 위시리스트(Wish) 에서 wished 속성 제거 - 위시 리스트 추가 여부는, 레코드가 있는지 여부로 판단 - Wish 엔티티는 숙소와 회원의 id 만 참조하도록 변경 * feat: 숙소제공항목(Provides) null 가능 항목 Wrapper 클래스로 변경 * style: 예약(Reservation) 테이블에 NOT NULL 항목 위로 이동 * feat: 각 도메인 필수항목 NOT NULL 설정 * fix: 불필요한 항목 제거 - 미완성된 FeesPolicy 커밋된 내역 제거 * fix: MySQL 주석으로 변경 * refactor: Accommodation 관련 도메인 클래스들 domain 패키지로 이동 * build: mysql, h2 의존성 참조 시점 수정 - mysql : 런타임에만 참조 - h2 : 테스트 런타임에만 참조 * build: queryDSL 의존성 및 설정 추가 - test/resources/application.properties : 테스트 환경 분리를 위해 추가 - QueryDslConfig : JPAQueryFactory 빈 등록 * refactor: QueryDSL 구현 편의를 위해, Money 타입 제거 * refactor: Accommodation 통계 쿼리 Repository 및 테스트 코드 추가 * git: QueryDLS QClass 경로 gitignore 추가 * refactor: Accommodation 의 Repository 관련 클래스들 repository 패키지로 이동 * refactor: AccommodationRepositoryCustom 테스트 클래스 이름 변경 * feat: 숙소 가격 통계가 입력값으로 주어진 단위대로 출력되도록 변경 * refactor: 숙소 가격 통계 쿼리 리펙토링 * feat: 숙소 가격 통계 서비스 추가 - AccommodationStatisticsService : 통계 서비스 제공, 가격 통계는 하루단위 캐싱 - AmountUnitPolicy : 얼마 단위로 가격 통계를 낼것인지의 기준 * refactor: AccommodationRepositoryCustom 을 더 명확한 이름인 AccommodationQueryRepository 로 변경 - 아직은 Interface 가 필요하지 않아 제거하였음 * feat: 숙소 가격 통계 API 추가 * feat: 숙소 가격 통계 API 추가 - API 문서 추가 - github action API 문서화 확인 단계 추가 * build: Github action API 문서 빌드 확인 step 추가 * 1차 배포 PR (#19) * Initial commit * build: Github action 백엔 개발 CI 에 working-directory 추가 * build: Github action 백엔드 운영 CI/CD 에 working-directory 추가 * feat: 전역 예외 클래스 추가 비즈니스 예외 클래스들이 HttpMethod 를 반환하도록 하여, 각 예외에 맞는 HTTP 상태코드를 반환할 수 있도록 함 * feat: 전역 예외 로직 추가 프로젝트 전역에서 발생하는 예외를 처리하는 로직 추가 - 비즈니스 예외 발생시에는 예외 클래스에 있는 HTTP 상태코드 반환 - 나머지 분류되지 않은 예외는 INTERNAL_SERVER_ERROR 로 반환 * feat: DB 연결 설정 추가 * feat: airbnb 테이블 정의(DDL) 추가 * feat: airbnb Entity 추가 * feat: 숙소(Accommodation) 에 가격 추가 * feat: 위시리스트(Wish) 에서 wished 속성 제거 - 위시 리스트 추가 여부는, 레코드가 있는지 여부로 판단 - Wish 엔티티는 숙소와 회원의 id 만 참조하도록 변경 * feat: 숙소제공항목(Provides) null 가능 항목 Wrapper 클래스로 변경 * style: 예약(Reservation) 테이블에 NOT NULL 항목 위로 이동 * feat: 각 도메인 필수항목 NOT NULL 설정 * fix: 불필요한 항목 제거 - 미완성된 FeesPolicy 커밋된 내역 제거 * fix: MySQL 주석으로 변경 * refactor: Accommodation 관련 도메인 클래스들 domain 패키지로 이동 * build: mysql, h2 의존성 참조 시점 수정 - mysql : 런타임에만 참조 - h2 : 테스트 런타임에만 참조 * build: queryDSL 의존성 및 설정 추가 - test/resources/application.properties : 테스트 환경 분리를 위해 추가 - QueryDslConfig : JPAQueryFactory 빈 등록 * refactor: QueryDSL 구현 편의를 위해, Money 타입 제거 * refactor: Accommodation 통계 쿼리 Repository 및 테스트 코드 추가 * git: QueryDLS QClass 경로 gitignore 추가 * refactor: Accommodation 의 Repository 관련 클래스들 repository 패키지로 이동 * refactor: AccommodationRepositoryCustom 테스트 클래스 이름 변경 * feat: 숙소 가격 통계가 입력값으로 주어진 단위대로 출력되도록 변경 * refactor: 숙소 가격 통계 쿼리 리펙토링 * feat: 숙소 가격 통계 서비스 추가 - AccommodationStatisticsService : 통계 서비스 제공, 가격 통계는 하루단위 캐싱 - AmountUnitPolicy : 얼마 단위로 가격 통계를 낼것인지의 기준 * refactor: AccommodationRepositoryCustom 을 더 명확한 이름인 AccommodationQueryRepository 로 변경 - 아직은 Interface 가 필요하지 않아 제거하였음 * feat: 숙소 가격 통계 API 추가 * feat: 숙소 가격 통계 API 추가 - API 문서 추가 - github action API 문서화 확인 단계 추가 * build: Github action API 문서 빌드 확인 step 추가 * fix: Github action API 문서 빌드 확인 step 타겟 디렉토리 수정 * fix: Github action API 문서 빌드 확인 step 타겟 디렉토리 수정 * fix: Github action API 문서 재빌드 스텝 수정 * fix: Github action API 문서 재빌드 스텝 수정 * build: 암호화 테스트 코드 활성화 * build: 운영 배포 환경 설정 추가 - application-prod.properties : 민감정보 암호화 하여 운영서버 설정 추가 - Dockerfile : prod 프로파일로 실행되도록 변경 - deploy.sh : docker 실행 로그 남기도록 변경 * 1차 배포 수정 (#20) * Initial commit * build: Github action 백엔 개발 CI 에 working-directory 추가 * build: Github action 백엔드 운영 CI/CD 에 working-directory 추가 * feat: 전역 예외 클래스 추가 비즈니스 예외 클래스들이 HttpMethod 를 반환하도록 하여, 각 예외에 맞는 HTTP 상태코드를 반환할 수 있도록 함 * feat: 전역 예외 로직 추가 프로젝트 전역에서 발생하는 예외를 처리하는 로직 추가 - 비즈니스 예외 발생시에는 예외 클래스에 있는 HTTP 상태코드 반환 - 나머지 분류되지 않은 예외는 INTERNAL_SERVER_ERROR 로 반환 * feat: DB 연결 설정 추가 * feat: airbnb 테이블 정의(DDL) 추가 * feat: airbnb Entity 추가 * feat: 숙소(Accommodation) 에 가격 추가 * feat: 위시리스트(Wish) 에서 wished 속성 제거 - 위시 리스트 추가 여부는, 레코드가 있는지 여부로 판단 - Wish 엔티티는 숙소와 회원의 id 만 참조하도록 변경 * feat: 숙소제공항목(Provides) null 가능 항목 Wrapper 클래스로 변경 * style: 예약(Reservation) 테이블에 NOT NULL 항목 위로 이동 * feat: 각 도메인 필수항목 NOT NULL 설정 * fix: 불필요한 항목 제거 - 미완성된 FeesPolicy 커밋된 내역 제거 * fix: MySQL 주석으로 변경 * refactor: Accommodation 관련 도메인 클래스들 domain 패키지로 이동 * build: mysql, h2 의존성 참조 시점 수정 - mysql : 런타임에만 참조 - h2 : 테스트 런타임에만 참조 * build: queryDSL 의존성 및 설정 추가 - test/resources/application.properties : 테스트 환경 분리를 위해 추가 - QueryDslConfig : JPAQueryFactory 빈 등록 * refactor: QueryDSL 구현 편의를 위해, Money 타입 제거 * refactor: Accommodation 통계 쿼리 Repository 및 테스트 코드 추가 * git: QueryDLS QClass 경로 gitignore 추가 * refactor: Accommodation 의 Repository 관련 클래스들 repository 패키지로 이동 * refactor: AccommodationRepositoryCustom 테스트 클래스 이름 변경 * feat: 숙소 가격 통계가 입력값으로 주어진 단위대로 출력되도록 변경 * refactor: 숙소 가격 통계 쿼리 리펙토링 * feat: 숙소 가격 통계 서비스 추가 - AccommodationStatisticsService : 통계 서비스 제공, 가격 통계는 하루단위 캐싱 - AmountUnitPolicy : 얼마 단위로 가격 통계를 낼것인지의 기준 * refactor: AccommodationRepositoryCustom 을 더 명확한 이름인 AccommodationQueryRepository 로 변경 - 아직은 Interface 가 필요하지 않아 제거하였음 * feat: 숙소 가격 통계 API 추가 * feat: 숙소 가격 통계 API 추가 - API 문서 추가 - github action API 문서화 확인 단계 추가 * build: Github action API 문서 빌드 확인 step 추가 * fix: Github action API 문서 빌드 확인 step 타겟 디렉토리 수정 * fix: Github action API 문서 재빌드 스텝 수정 * build: 암호화 테스트 코드 활성화 * build: 운영 배포 환경 설정 추가 - application-prod.properties : 민감정보 암호화 하여 운영서버 설정 추가 - Dockerfile : prod 프로파일로 실행되도록 변경 - deploy.sh : docker 실행 로그 남기도록 변경 * fix: 운영 DB URL 수정 * docs: API 문서 오타 수정 * feat: 숙소 가격 통계 API 가 더미 데이터를 반환하도록 변경 * feat: 숙소 가격 통계 API 에 전체 숙소 평균 가격 추가 * docs: 백엔드 README 추가 * Update README.md * Update README.md * Update README.md * [TEAM-25] Airbnb 2주차 1번째 PR Nick + Benny (codesquad-members-2022#169) * Update issue templates * Update issue templates * Update issue templates * Update issue templates * [FE] 체크인, 체크아웃 기능 구현 완료 (#25) * Initialize project using Create React App * feat: 개발 환경 셋팅 create-react-app(typescript) styled-components(typescript) styled-reset eslint prettier material ui * refactor: 개발 환경 셋팅 수정 * feat: 디렉터리, 폰트, styled-components 환경 구성 global.js 추가 theme.js 추가 google fonts 추가 * feat: 절대경로 설정 추가 * feat: router 설정 * feate: components에 컴포넌트별 디렉터리 생성 * feat: Hero 컴포넌트 마크업, 스타일 추가 * feat: constant 디렉터리, constant.js 파일 추가 * feat: utils.js 파일 추가 * design: utils.js WrapperMarginCenter 수정 * feat: 중첩 router 사용해 공통 header 추가 * design: Hero, Home 컴포넌트 스타일 수정 * feat: GNB 컴포넌트 마크업, 스타일 추가 * refactor: 코드리뷰 피드백 수정 * refactor: Hero, GNB 마크업 스타일 수정 * feat: header 마크업, 스타일 추가 * remove: js 파일 제거 * refactor: header 마크업 스타일 리팩토링 * feat: search 마크업, 스타일 추가 * style: favicon 추가 * feat: modal portal 기능 추가 * refactor: type들 파일로 따로 분류 * feat: modal backdrop show, hide 기능 추가 * feat: 캘린더 모달 내 캘린더 버튼 추가 * feat: Calendar 년, 월에 따른 일수 생성 기능 추가 * feat: 캐러셀 움직이는 기능 추가 * feat: 캘린더 새로운 년, 월 추가하는 기능 추가 * feat: 캘린더 마지막 요소일때만 추가하는 기능 오른쪽 버튼을 누를때 보여지는 요소가 캘린더의 마지막 아이템일경우에만 캘 린더를 추가하는 기능 추가 * style: 불필요한 주석 제거 * style: 캘린더 '일'에 마우스 호버시 스타일 추가 color, background-color 변경 * refactor: 중복되는 함수, type 변경 * feat: 과거 일자 스타일, 클릭 방지 기능 추가 * fix: 캘린더에서 오늘 날짜를 비활성시키는 버그 픽스 * feat: DateCell 컴포넌트 추가 * refactor: 코드리뷰 피드백 수정 * style: CalendarModal 변수명 리네임, 상수 추가 * refactor: CalendarModal reducer 부분 리팩토링 * rename: 파일들 FE 폴더로 이동 * feat: 라우터를 이용시 공통으로 보이는 배경 수정 * feat: 캘린더 요소 클릭시 선택되는 기능 구현 * fix: 체크인 관련 버그 수정 체크인을 누른 후 체크인보다 작은 날 누를시 생기는 버그 수정 * feat: 캘린더 체크인, 체크아웃 기능 구현 완료 Co-authored-by: jinan159 <wlsdhks0423@naver.com> * refactor: 새로운 Wish 를 만드는 정적 팩토리 메소드 추가 * refactor: Accommodation, Member, Reservation 에 정적 팩토리 메소드 추가 새로 추가할 엔티티를 만들 경우에 사용 * feat: WishRepository 에 숙소 아이디와 회원 아이디로 위시 리스트를 조회하는 기능 추가 * feat: 예약 가능 숙소 정보와 위시 리스트 정보를 병합하여 반환하는 서비스 추가 * feat: 예약 가능 숙소 정보와 위시 리스트 정보를 병합하여 반환하는 API 추가 * fix: Github action API 문서 빌드 확인 step 타겟 디렉토리 수정 * fix: Github action API 문서 재빌드 스텝 수정 * 가격 통계 API 더미 데이터 반환 배포 PR (#23) * docs: API 문서 오타 수정 * feat: 숙소 가격 통계 API 가 더미 데이터를 반환하도록 변경 * feat: 숙소 가격 통계 API 에 전체 숙소 평균 가격 추가 * 조회 API 배포 (#45) * feat: 숙소 가격 통계 API URL 복수형으로 변경 * feat: 숙소제공항목 row 로 저장 및 명칭 단수로 변경 - Provides -> Provide - AccommodationProvide 추가 - 숙소제공항목을 enum 으로 관리 - enum 을 column 으로 변환하는 ProvideElementsConverter 추가 * feat: 변경 불가능한 컬렉션 대신, 복사하여 반환하도록 변경 * style: static 변수 대문자로 변경 * style: import * 대신, 필요한 것만 import 하도록 변경 * feat: 시작 종료일자 검증로직 추가 * feat: AccommodationRepository, ReservationRepository 추가 * feat: Accommodation 생성자 추가 - @AllArgsConstructor 추가 - @entity 를 위해 기본 생성자를 @NoArgsConstructor 로 변경 * refactor: ProvideElement -> ProvideName 으로 명칭 변경 - ProvideElement -> ProvideName - ProvideElementConverter -> ProvideNameConverter * feat: Schedule 에 정적 생성 메소드 추가 * feat: Reservation 에 Getter 및 AllArgsConstructor 추가 * feat: 예약 가능한 숙소 조회 쿼리 추가 * feat: 예약 일정을 저장하는 변수 타입 변경 (LocalDateTime -> LocalDate) * feat: 예약 가능한 숙소 조회 서비스 추가 * build: MySQL Point 를 처리를 위해 hibernate-spatial 의존성 추가 및 Dialect 설정 * refactor: 전역 예외처리 로직 에러 로깅 추가 * refactor: Accommodation 의 Point 의존성 변경 * refactor: StartEndDate null 방어코드 추가 * refactor: Address toString 추가 * refactor: AccommodationService 에서 조회시 Lazy loading 되도록 수정 * style: 미사용 import 제거 * refactor: 일급컬렉션 객체 제거 * feat: 예약 가능한 숙소 조회 API 추가 * style: 불필요한 공백 제거 Co-authored-by: Ubeom Kim <kimub1204@gmail.com> Co-authored-by: Nick <75640166+decay95@users.noreply.github.com> Co-authored-by: Eunji Lee <62249058+nanjeong@users.noreply.github.com> Co-authored-by: Benny <77577858+uBeom@users.noreply.github.com>
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요, Q!
이번 리뷰도 잘 부탁드립니다!
2주차 화요일 진행상황
이번 PR에서는 캘린더를 거의 다 구현했는데 저번주와 마찬가지로 typesript를 사용하면서 나타나는 여러가지 오류와 캘린더를 구현하는게 어려워서 개발 속도가 느려졌습니다. 그리고 차트를 그리기 위해 canvas를 학습 중에 있습니다.