[Team 26-FE] 1주차 01 PR요청 (이든, 도리) #22
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
Co-authored-by: HongJungKim(이든) <HongJungKim-dev@users.noreply.github.com>
Co-authored-by: HongJungKim(이든) <HongJungKim-dev@users.noreply.github.com>
- no-vars - no-require
[Build] 개발환경 세팅
kimyouknow
changed the title
guswns1659
pushed a commit
that referenced
this pull request
May 25, 2022
deprecated-hongbiii
approved these changes
May 25, 2022
There was a problem hiding this comment.
이든, 도리 안녕하세요!
3주간 함께할 리뷰어 어텀입니다. 😀
남겨주신 링크 문서 다 읽어봤는데 정리를 굉장히 잘하고 계시네요! 앞으로도 꾸준히 기록 남기셨으면 좋겠어요. :)
사용자 스토리 작성, 작업 시간 측정도 좋고 특히 코딩 마라톤 재밌게 봤습니당 ㅋㅋㅋ
요즘은 webpack 대신 vite를 사용하는 곳도 점차 늘어가는 추세인데요,
관심 있으시다면 다음 프로젝트나 수료 후에 진행할 프로젝트에 적용해보셔도 좋을 것 같아요!
API 설계에서 하나, 프로젝트 디렉토리 구조에서 하나 피드백 드릴 것이 있습니다!
API 설계
전반적으로 잘 구성해주셨는데 GET /search params의 location에 조금 보완이 필요해보여요.
지도에 보여지는 영역이 요청에 들어가야 해서 좌표가 2개 필요하겠습니다!
실제로 에어비앤비 검색 params를 보시면 우상단, 좌하단의 좌표를 함께 보내고 있어요.
- ne_lat
- ne_lng
- sw_lat
- sw_lng
디렉토리 구조
config라는 디렉토리 아래에 webpack 관련 config들이 들어있는데요, FE 디렉토리 하위에 eslint, prettier, babel에 대한 config 파일들이 있어서 정리가 덜 된 느낌이에요.
보통 config 파일은 프로젝트 루트 경로(여기서는 FE)에 두긴 하는데 config 디렉토리로 관리하기로 했다면 모든 config 파일을 config 디렉토리에 넣어주는 게 좋을 것 같습니다. 😀
내일 바로 이어서 작업하셔야 하니 머지 바로 할게요!
Comment on lines
+24
to
+30
| ## 원인 | ||
|
|
||
| ex) | ||
| 1. Go to '...' | ||
| 2. Click on '....' | ||
| 3. Scroll down to '....' | ||
| 4. See error |
There was a problem hiding this comment.
버그를 등록할 때에는 원인까지 파악하기는 힘들어보여요.
원인은 이슈에 코멘트로 달아두어 히스토리를 남기는 게 좋을 것 같습니다. :)
(+)
재현되는 과정, 사용 기기와 OS 및 브라우저의 버전은 버그 리포팅에 필수입니다!
템플릿에 추가 부탁드려용~~
|
아! 그리고 두 분은 지난 프로젝트에서 타입스크립트를 사용하셨었는지 궁금해요. 필수 선택 미션{아래 미션 중 한가지를 필수로 선택해서 개발해야 함}
|
jminie-o8o
pushed a commit
that referenced
this pull request
May 27, 2022
…endar feat: 요구사항에 맞춰 달력 구현(오늘 날짜 이전 선택 불가 및 dialongRange로 구현
guswns1659
pushed a commit
that referenced
this pull request
May 29, 2022
- Accommodation의 연관 엔티티 매핑 Address, Location
guswns1659
pushed a commit
that referenced
this pull request
May 29, 2022
- Room, City, AccommodationType 매핑
GangWoon
pushed a commit
that referenced
this pull request
May 31, 2022
[iOS] 특정 시점을 입력하면 DayCreator가 해당 달의 �달력 날짜 정보를 반환함
wnsxor1993
pushed a commit
that referenced
this pull request
Jun 3, 2022
…#22) Date 표기방법을 변환하는 객체로써의 이름을 가지도록, DateConverter라는 이름으로 구조체 이름 변경
wnsxor1993
pushed a commit
that referenced
this pull request
Jun 3, 2022
junseokseo9306
added a commit
that referenced
this pull request
Jun 8, 2022
Feature/android/UI/calendar
hayoung123
pushed a commit
that referenced
this pull request
Jun 11, 2022
* refactor : Calendar 컴포넌트 분리 Co-authored-by: JinJeon <JinJeon@users.noreply.github.com> * feat : 캘린더 슬라이더 구현 (#18) * refactor : 캘린더 스타일 지정 위치 변경 * test : slide 기능 구현 위해 테스트 코드 작성 * feat : animation으로 일회성 슬라이딩 구현 * feat : 캘린더 슬라이드 기능 구현 * refactor : 특정 간격에 대한 월, 연도를 구하는 형식으로 변경 * refactor : 연도, 월 계산 함수 통해 이벤트 함수 통합 * fix : 1월이 13월으로 나오는 버그 해결 * fix : 월 표시 문제 수정 * refactor : 스타일 컴포넌트 및 적용된 컴포넌트 이름 수정, 적용 * refactor : 연도, 월 표시 부분 리펙토링 * refactor : CalCell 부분 이름 변경 및 적용 * fix : 검색창, 모달창에 해당 월이 한 달 적게 표시되는 문제 해결 * fix : 달력 마지막 날이 토요일에 걸리는 경우 그라데이션 안되는 오류 해결 * feat : 반응형 미니바 구현 및 모달 외부 클릭 시 창 닫기 기능 구현 (#22) * feat : MiniSearchBar 컴포넌트 생성 * feat : 미니바 애니메이션 구현 * feat : 미니바 애니메이션 * fix : 미니바 position 수정 * fix : 미니바 애니메이션 수정 * feat : StyledSearchBarWrapper 로 미니바, 서치바 랩핑 * refactor : props 받아오는 것 수정 * refactor : 미니바에 context를 사용하여 정보 표시 * feat : Modal 에 Background 추가 * feat : modal 외부 클릭 시 modal 창 닫히는 기능 구현 * fix : 모달 클릭 시에도 모달이 꺼지는 문제 해결 모달 자체에 onClick * feat : Portal 을 사용하여 모달 창 닫기 기능 구현 * fix : portal 사용 시 모달 레이아웃이 깨지는 문제 해결 box-sizing: border-box; 을 추가했다. * feat : 검색 버튼 클릭 시 모달 닫기 * fix : 모달 Background 로 인한 GNB z-index 수정 * fix : 모달 Background 로 인한 GNB z-index 수정 테스트를 위한 border 삭제 * feat : 검색 후 숙소 리스트 구현 (#24) * feat : 이미지 및 데이터 추가 및 위치 이동 * feat : hotels api 추가 및 수정 * fix : 이미지, api 이동에 따른 파일 수정 * feat : Search Desc의 필터된 정보란 설정 * feat : 검색 결과 상세값 및 가격 불러오기 일부 설정 * feat : 결과창 금액란 생성 * feat : 숙소별 이미지를 불러오기 위한 api 생성 및 적용 * refactor : 페이지 수 지정 * feat : 반응형 미니바 구현 완성 (#25) * feat : 미니바 slide-in 애니메이션 추가 Co-authored-by: JinJeon <JinJeon@users.noreply.github.com> * fix : 반응형 미니바 구현 완료 검색바 외부를 클릭하면 반응형으로 검색바를 미니바로 바꾼다. * refactor : miniSearchBarIsHidden & searBarIsHidden 상태를 selectedSearchBar 상태 하나로 통합 Co-authored-by: JinJeon <JinJeon@users.noreply.github.com> * feat : 검색 결과 지도에 표시 (#26) * feat : map 생성 및 컴포넌트 분리를 위한 상태 위치 변경 * feat : 지도 영역 분리 및 데이터에 따른 지도 표시 구현 * feat : 검색 지도가 스크롤 시 고정이 유지되도록 설정 * fix : 기타 에러 사항 수정 (#27) * fix : 캘린더 hover 시 색상 변경되도록 수정 * fix : PR 내용 반영 및 수정 Co-authored-by: Serin-Kim <serin9864@gmail.com> Co-authored-by: JinJeon <JinJeon@users.noreply.github.com> Co-authored-by: Serin Kim <68533016+Serin-Kim@users.noreply.github.com>
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.
안녕하세요 어텀. 도리 이든입니다. 잘부탁드립니다!
고민했던 점
1. 그라운드룰 작성
3주간 프로젝트를 진행할 때 지켜야할 룰을 정했습니다. wiki링크
2. 협업 전략 정하기
협업을 위한 서비스로 노션과 깃허브를 선택했습니다.
깃허브 전략, PR리뷰 및 컨벤션, 깃허브 컨벤션, 코드 컨벤션 등을 정리해 wiki에 정리했습니다.
3. 미션 요구 사항 분석
사용자 스토리를 중심으로 요구사항을 분석한 후 api설계를 진행했습니다.
4. 개발 환경 세팅
cra를 사용하지 않고 webpack을 활용해 react + typescript 개발환경을 세팅했습니다.
진행상황
이전 미션에서 사용했던 모달 컴포넌트를 페어 수정하여 npm에 배포한 뒤, 이번 프로젝트에 받아서 사용할 예정입니다. (npm에 배포할 레포지토리 링크)
수정할 부분: typescript 리팩토링 및 로직 수정(열고 닫는 로직도 한 파일에 모아두기, createPortal 장단점 파악)