-
Notifications
You must be signed in to change notification settings - Fork 79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TEAM-25] Airbnb 3주차 2번째 PR Nick + Benny #301
Open
decay95
wants to merge
86
commits into
codesquad-members-2022:team-25
Choose a base branch
from
jinan159:FE
base: team-25
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
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
feat: modal portal 기능 추가
오른쪽 버튼을 누를때 보여지는 요소가 캘린더의 마지막 아이템일경우에만 캘 린더를 추가하는 기능 추가
color, background-color 변경
* 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: 캘린더 체크인, 체크아웃 기능 구현 완료 * [FE] 체크인, 체크아웃 기능 구현 (#24) * style: CalendarModal 변수명 리네임, 상수 추가 * refactor: CalendarModal reducer 부분 리팩토링 * rename: 파일들 FE 폴더로 이동 * feat: 라우터를 이용시 공통으로 보이는 배경 수정 * feat: 캘린더 요소 클릭시 선택되는 기능 구현 * [FE] footer 레이아웃 구현 완료 (#27) * docs: footer 이미지들 추가 * feat: footer 구현 완료 * [FE] 인원수 기능 구현 완료, 코드 전체적인 리팩토링 (#28) * feat: Personnel, PersonnelCounter 마크업, 스타일 * feat: 인원 종류 별 버튼 기능 구현 * feat: 버튼 클릭시 인원 추가, 감소 기능 추가 성인 인원수가 0일때, 어린이(유아) 인원을 추가할 경우 성인과 어린이(유아) 인원을 1씩 올리는 기능 추가 * feat: 버튼 이미지 변경 기능 추가 인원이 max, min 일경우 비활성화 이미지로 변경하는 기능 추가 * feat: 인원 입력시 초기화 버튼 생성 기능 추가 * feat: 초기화 버튼 클릭시 초기화 기능 추가 인원 초기화 버튼 클릭시 인원 초기화 기능 추가 * feat: 인원 버튼 누를시 인풋텍스트 변경 인원 추가, 감소 버튼 누를시 인원에 따른 인풋 텍스 트 변경 * refactor: 변수명 길이 줄임 Personnel, PersonnelCounter 컴포넌트에서 personnel Context.personnelState.adult, personnelContext.per sonnelState.child, personnelContext.personnelState .infant 들을 adult, child, infant로 비구조화를 통 해 변수명 길이 줄임 * refactor: Personnel 관련 컴포넌트 리팩토링 Personnel, PersonnelModal, PersonnelCounter 컴포넌 트 리팩토링 * refactor: Search 컴포넌트 리팩토링 * refactor: constant.ts 리팩토링 * refactor: 코드리뷰 피드백 리팩토링 * Calendar, CalendaModal 리팩토링을 하였고 CalendarMo dalBtns 컴포넌트를 만들어 CalendaModal 컴포넌트의 로직을 분리해주었습니다. * CustomDate 클래스, Dates 컴포넌트를 만들어 Calen dar 컴포넌트의 로직을 분리 해주었습니다. * DateCell의 컴포넌트 명을 DateElement로 변경하였 습니다. * Typescript 관련된 코드의 네이밍, 무분별한 interf ace의 사용을 변경 해주었습니다. * build: 프론트 배포 스크립트 추가 * build: github action 빌드 Container registry 변경 * build: github action 이미지 생성 로직 수정 * fix: github action docker build 경로 수정 * build: docker build 수정 Co-authored-by: Ubeom Kim <[email protected]> Co-authored-by: Benny <[email protected]>
* feat: 페이지 이동시 header 부분 변경 기능 추가 * style: rooms 페이지 레이아웃 구성중 PR 보내려고 커밋함
build: fe 자동배포 스크립트 추가
* feat: 페이지 이동시 header 부분 변경 기능 추가 * style: rooms 페이지 레이아웃 구성중 PR 보내려고 커밋함 * feat: skeleton UI 추가
추가로 확대, 축소, 좌표 정보 출력 기능 추가
[FE] 카카오 맵 숙소, 가격 표시 기능 추가
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!
마지막 PR입니다.
지난 수요일에 저희가 정신이 없어서 PR을 깜빡 잊고 못했습니다 죄송합니다 ㅠㅠ
리뷰 항상 감사했습니다. 수고하셨습니다!
진행사항
하필 개인 일정이 마지막에 많이 꼬여서 진행이 더뎠던 한 주 였습니다ㅠㅠ
상세 검색을 위한 기능들을 모두 구현한 뒤, 지도 api를 사용해 코드스쿼드를 중심으로 지정된 반경 내의 숙소 정보를 불러오는 기능까지 시도했습니다!
BE api 사용 시도를 했었는데, CORS 오류가 나타났는데 건강상의 문제로 BE멤버가 해결할 수 없는 상황이 되어서 Chrome 확장 프로그램을 이용해 로컬 환경에서 CORS를 해결하여 개발을 진행했습니다!
또, 곡선을 이용해 상세 요금 차트를 그리려 시도했는데, 아무래도 숙소 정보 수량이 적고, 범위가 작다보니 곡률이 너무 완만하고 가독성이 좋지 않아, 막대 차트로 변경하여 진행했습니다.
Canvas에 많이 익숙해질 수 있었던 것 같습니다.
이번 미션에서 협업과 분업을 어떻게 해야할지 헷갈리는 부분이 많아 서로 따로 구현한 것들이 많아 아쉬웠던 것 같습니다.
그래도 가능한 할 수 있는 것을 다 해본 것 같습니다!
추가 미션사항을 진행하지 못해 많이 아쉽습니다. 저희는 각자 미션이 아니더라도 개인적으로 간단한 api를 직접 만들어 볼 예정입니다!