[Team22][FE]1주차 금요일 pr #98
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
- 모달창 경로 수정 - styled-components와 mui 구분 - 그 외는 리뷰 참고 Co-authored-by: herrakam <herrakam@users.noreply.github.com>
* Design: 기존 스타일 수정 - 라우터에 따른 스타일 변경 추가 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Design: Result 페이지 UI 구현 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Rename: Hotel.tsx 파일 경로 이동 * Design: result 페이지의 숙소 리스트 컴포넌트와 일부 컴포넌트의 스타일 변경 * Design: Hotel 컴포넌트와 관련된 스켈레톤 UI 생성 * Refactoer: 서치바 리팩토링 - 각 컴포넌트를 공통된 스타일 컴포넌트로 변경 - 일부 태그 변경 - hover 이펙트 추가 Co-authored-by: herrakam <herrakam@users.noreply.github.com>
- devDependencies에 설치된 스타일 라이브러리를 dependencies로 재설치함
junzero741
reviewed
May 29, 2022
| <Divider orientation="vertical" /> | ||
| <Personnel /> | ||
| <SearchButton /> | ||
| <Divider orientation="vertical" sx={{ height: '60%' }} /> |
There was a problem hiding this comment.
Divider 의 sx 프로퍼티는 어떤 역할을 하나요? 조금 더 의미가 드러나게 이름을 지어도 좋을 것 같습니다.
There was a problem hiding this comment.
sx는 MUI에서 사용자 정의 스타일을 사용하기 위한 prop 입니다. 이번 저희 프로젝트에서 스타일 수정이 많이 필요한 mui 컴포넌트는 styled-component의 styled를 사용했고 이렇게 수정이 많이 필요하지 않은 경우는 sx prop을 사용했습니다!
junzero741
reviewed
May 29, 2022
Comment on lines
+52
to
-60
| export const InputState = styled.span` | ||
| width: 100%; | ||
| text-align: left; | ||
| color: ${({ theme }) => theme.colors.grey2}; | ||
| ${({ theme }) => theme.fontStyles.normal16px}; | ||
| text-align: left; | ||
| `; |
There was a problem hiding this comment.
리액트의 주요 컨셉 중 하나가 상태 변경에 따라 컴포넌트를 리렌더링한다는 것인데요.
변수명에 state(상태) 가 들어가면 '이 변수에 할당된 값이 변경되는 경우가 생기겠구나' 라고 오해할 여지가 있는 것 같습니다.
또, 그냥 변수도 아니고 컴포넌트 이름이 state를 포함하고 있으니 좀 더 어색하게 느껴지는 것 같아요..!
There was a problem hiding this comment.
컴포넌트에 맞는 이름으로 고치도록 하겠습니다!
junzero741
reviewed
May 29, 2022
Comment on lines
+1
to
+22
| import { Skeleton } from '@mui/material'; | ||
| import styled from 'styled-components'; | ||
|
|
||
| function SkeletonHotel() { | ||
| return ( | ||
| <SkeletonContainer> | ||
| <Skeleton variant="rectangular" animation="wave" width={330} height={200} /> | ||
| <div> | ||
| <Skeleton width={134} animation="wave" /> | ||
| <Skeleton width={268} animation="wave" /> | ||
| <Skeleton width={268} animation="wave" /> | ||
| </div> | ||
| </SkeletonContainer> | ||
| ); | ||
| } | ||
|
|
||
| const SkeletonContainer = styled.div` | ||
| display: flex; | ||
| gap: 24px; | ||
| `; | ||
|
|
||
| export default SkeletonHotel; |
junzero741
reviewed
May 30, 2022
| @@ -53,18 +48,27 @@ function Header() { | |||
| const HeaderWrap = styled.header<Position>` | |||
There was a problem hiding this comment.
Position 의 타입을 단순 string 으로 지정해놓으셨는데요.
물론 location.pathname 이 항상 문자열로 들어오는 건 맞습니다만,
제 생각엔 Position 타입의 범위를 좀 더 좁힐 수 있을 것 같아요.
type Position = {
position : "/" || "/result"
}
이런 식으로요!
물론 개발 과정에서 라우트가 추가된다면 그때마다 뒤에 || "/라우트명" 이라고 또 적어줘야 하는 복잡스러움은 있겠지만,
시스템 전체적으로 보았을 때 Position.position 에 이상한 문자열이 들어가는 걸 사전에 방지해주는 역할이 크다고 생각합니다!
junzero741
reviewed
May 30, 2022
There was a problem hiding this comment.
안녕하세요!!
PR 이 올라오지 않아서 아직 코드 양이 많지 않으신가보다 했는데,
깜빡하고 리뷰어 지정을 안하셨다는 연락을 받았습니다. ㅎㅎㅎ
제가 코쿼 학생일 때 코드량이 많지 않아서 PR을 안보낸 적도 많았거든요..ㅎㅎ
PR을 보니 여러분은 제 경우와 다른 경우지만, 만에 하나 코딩을 �한 줄도 못 하셨더라도
궁금한 점, 얘기해보고 싶은 점을 주로 적어서 PR 을 날려주시면 같이 얘기 나누는 것도 너무 좋은 것 같습니다!
PR 을 늦게 보낸건 너무 염려마셨으면 좋겠습니다. ㅎㅎ (깜빡할수도 있죠 저흰 사람인걸요 ㅜㅜ)
무튼 지난 한 주 고생 많으셨고, 이번 주도 파이팅하셔요!!!
컨텍스트를 구현할 예정인데, 자료를 찾아보니 State와 dispatch를 구분해서 적용하는 코드가 대부분이었습니다. 구분해서 적용하는 방법과 통합해서 적용하는 방법중 어떤 방법이 효율적인지 궁금합니다!!
ContextAPI 를 직접 구현할 예정이라는 말씀이시죠? state 와 dispatch를 구분해서 적용하는 방법을 사람들이 주로 쓰는 이유는 무엇일까요? 또 대부분이 구분해서 적용하지만 통합해서 적용하고 싶어진 이유는 무엇인가요? 이 부분에 대해 같이 고민해보고 효율성을 얘기해보면 좋을 것 같습니다.
|
ContextAPI를 직접 구현하는 것은 아니고 사용한다는 이야기였는데 저희가 잘못전달한것 같습니다ㅎㅎ 사람들이 state와 dispatch를 나누는 이유는 state는 자주 변경되지만 dispath는 변경되는 일이 없어서 리렌더링을 방지하기 위해서 분리를 했다고 합니다. 저희는 이 점까지 고려를 못했는데 이유를 알고나니 분리하기로 결정했습니다!! |
bugpigg
referenced
this pull request
in bugpigg/airbnb
Jun 7, 2022
* docs: README.MD 프로젝트 안내 내용 추가 * [#3] Feat: 스프링부트 프로젝트 초기 설정 * build : 스타일링 관련 초반 세팅 material 페키지 설치 globalStyle 설정 아이콘, 배경 다운로드 * Build: 폰트와 관련된 theme 설정 추가 * build : app.js에 themeProvider, GlobalStyles 적용 * build : index.css 삭제 * Bulid: react-router-dom 설치 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Style: 사용하지 않은 파일 import 경로 삭제 * Feat: 라우터 적용 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Refact: 라우터와 관련된 컴포넌트를 파일로 분리 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * 컨트롤러 레이어 초기 구현 (street62#11) * [#4] Feat: 숙소 검색 결과 반환 컨트롤러 생성 - 숙소 검색 쿼리 파라미터를 받아 해당하는 숙소의 검색 결과를 반환하는 컨트롤러를 생성하였습니다. * [#4] Feat: 숙소 검색 결과 반환 DTO 생성 - 숙소 검색 결과 DTO 를 생성하였습니다. * [#4] Feat: 숙소 예약 폼 반환 컨트롤러 작성 - AccommodationController의 getReseverForm() 메서드로 숙소 예약 폼 반환. * [#4] Feat: 숙소 예약 폼 Response DTO 작성 - ReserveFormResponseDto를 작성해 숙소 예약 폼 GET 요청 시 반환되는 DTO 작성. * [#4] Feat: 숙소 예약 컨트롤러 작성 - AccommodationController의 postReservation() 메서드로 숙소 예약 요청. * [#4] Feat: 숙소 예약 Request DTO 작성 - ReserveRequestDto를 작성해 숙소 예약 POST 요청시 DTO 역할을 수행. * [#4] Fix: 숙소 검색 결과 반환 컨트롤러 수정 - 기존의 SearchQueryResponseDto 객체로 Response를 반환하던 형식을, SearchQueryResponseDto 리스트 형식으로 반환하게 수정. * [#4] Fix: 숙소 예약 폼 반환 컨트롤러 수정 - 반환형 제너릭 DTO 수정. Co-authored-by: Senglee Choi <tmdfl1027@gmail.com> * Build: 린트 설정 추가 - plugin:react/jsx-runtime 추가 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Refactor: MUI의 CssBaseline, ThemeProvider 적용 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Build: Svg 파일 import를 위한 custom.d.ts 파일 생성 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Design/Feat: 헤더 UI 구현 및 result 페이지의 일부 기능 추가 - result 페이지의 미니 서치바를 클릭하면 큰 서치바로 변함 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Feat : 서치바에서 마우스 나갈 시 미니 서치바로 바뀌는 기능 구현 * Feat : 각 항목당 모달 생성 * 검색 결과 조회 Service 로직 구현 (street62#14) * [street62#12] Feat: 숙소 검색 Service 로직 설계 - AccommodationService 클래스 작성 - 숙소 검색 로직 설계. DB에는 체크인 날짜를 기준으로 한 번 접근하고, 이후에는 자바 코드로 필터링하도록 구현. * [street62#12] Feat: AccommodationRepository 클래스 추가 - 추후 JPA와 연동하여 설계 예정. Co-authored-by: Senglee Choi <tmdfl1027@gmail.com> * 숙소 도메인 구현 (street62#17) * [street62#13] Feat: Accommdation 도메인 추가 - Accommodation 도메인 클래스를 추가하였습니다. * [street62#13] Fix: AccommodationService 오타 수정 - AccommodationService 클래스의 오타수정을 하였습니다. * Design: 인원, 요금 모달창 구현 - 기능은 작동하지 않음 - 요금 범위 슬라이더는 아직 구현하지 않음 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * 검색 요청 DTO 추가 및 쿼리 파리미터 DTO 매핑 (street62#21) * [street62#18] Feat: 검색 요청 DTO 추가 SearchQueryRequestDto 추가. * [street62#18] Refactor: 쿼리 파라미터를 SearchQueryRequestDto로 매핑 @ModelAttribute를 사용해 검색 시 들어오는 쿼리 파라미터를 Dto로 바로 매핑. 이전에 Map<String, String>으로 받아 자료형 변환하던 코드를 삭제함. * [street62#18] Feat: 리퀘스트 DTO 추가에 따른 SearchQueryResponseDto 변경 of() 메서드를 추가해 서비스에서 Accommodation을 응답 DTO로 변환할 수 있도록 함. * [street62#16] Feat: 체크인, 체크아웃 날짜 사이에 이용 가능한 숙소 검색 메서드 추가 (street62#22) findByDateBetween() 추가 * 리팩토링 (street62#24) * [street62#23] Style: DTO 클래스 코드 포맷팅 수정 - DTO 클래스의 코드 포맷팅을 수정하였습니다. * [street62#23] Style: AccommodationController 클래스의 코드 포맷팅 수정 * [street62#23] Refactor: 서비스 및 레포지토리 클래스 리팩토링 - AccommodationService 클래스 코드 포맷팅 수정 및 예약 날짜 필터링 로직 추가하였습니다. - AccommodationRepository 클래스 코드 포맷팅 및 정의한 findByDateBetween() 메서드를 제거하였습니다. 이는 자바 코드 기반으로 예약 날짜를 필터링하게 수정하였기 때문입니다. * [street62#23] Refactor: Accommodation 도메인 클래스 리팩토링 - Accommodation 클래스에 예약 날짜 필터링 메소드 추가 및 코드 포맷팅을 수정하였습니다. * [street62#23] Refactor: PeopleConstraint 클래스 추가 - 기존의 Map 컬렉션으로 관리되었던 수용인원 정보에 대하여,PeopleConstraint 클래스를 통해 관리되도록 리팩토링 하였습니다. * Design: 메인 페이지의 이미지 추가 * Design: location.pathname에 따라 헤더의 position 변경 * Style: 테스트코드 삭제 * Design: 헤더 스타일 일부 수정 - 서치바의 배경색 추가 - PriceModal을 display: none으로 설정 * Style: html의 lang을 ko로 변경 * Refactor: 헤더의 메뉴 태그를 MUI의 Link로 변경 * Refactor: 로고와 마이페이지 버튼 개선 - 로고를 클릭하면 메인페이지로 이동 - 마이페이지 아이콘들을 버튼 태그안에 넣음 * Build: material-ui 제거 후 MUI 재설치 * Design: 이미지 위치 조정 * Refactor: aria-label 추가 * Accommodation 위도/경도 필드 수정 (street62#27) * Refactor: 각 숙소별 위도, 경도 값의 자료형 double 로 변경 * Refactor: SearchQueryResponseDto 에 위도, 경도값 추가 * 리뷰어님의 코멘트 반영 (street62#30) * Refactor: 검색 결과 GET 메서드 {query} 삭제 PathVariable이 아닌 쿼리 파라미터로 들어오는 값을 ModelAttribute 애노테이션으로 매핑하므로 해당 내용 불필요. * Refactor: ReserveRequestDto Setter 삭제 및 생성자 추가 @requestbody로 객체 매핑하므로 Setter는 불필요. 생성자를 통해 객체와 매핑할 수 있도록 함. * Refactor: SearchQueryResponseDto 생성자 추가 및 of() 메서드 수정 AllArgsConstructor 추가하여 모든 매개변수를 받는 생성자를 두고, of() 메서드에서 빌더를 호출해 DTO 객체를 반환하도록 함. * Refactor: REST API 컨벤션에 맞게 URL 수정 - AccommodationController 의 매핑 URL 을 REST API 컨벤션에 맞게 수정하였습니다. * Refactor: Reservation 객체 생성을 통한 Accommodation 리팩토링 - Reservation 객체를 생성하여 체크인, 체크아웃 날짜를 관리하게 하였습니다. Co-authored-by: Senglee Choi <tmdfl1027@gmail.com> * Feat: Reservation 클래스 추가 - Accommodation 엔티티의 체크인, 체크아웃 날짜를 관리하는 Reservation 클래스 추가 * Feat: 전체 숙소의 가격 반환하는 API 구현 (street62#32) 전체 숙소의 가격을 List<Integer>로 반환하는 API 구현 리포지토리 로직은 @query 애노테이션으로 모든 숙소의 1박 가격 컬럼을 SELECT 해 오는 방식으로 구현함. * 도메인 관련 클래스 JPA 엔티티로 수정 및 새로운 엔티티 추가 (street62#36) * Feat: 도메인 객체 JPA 엔티티로 변환 기존의 Accommodation, Reservation을 JPA 엔티티로 변환. * Feat: User 엔티티 객체 추가 Reservation의 예약자를 의미하는 User 엔티티 객체 추가. Reservation - User는 단방향 N:1 관계로 Reservation에서 JoinColumn을 통해 연관관계 관리. * Feat: Reservation 누락된 필드 추가 Reservation 객체의 필드를 ERD와 비교하여 누락된 부분 추가. Co-authored-by: Senglee Choi <tmdfl1027@gmail.com> * 숙소 검색 서비스 로직을 DB 쿼리문으로 변경 (street62#38) * Feat: AccommodationRepository 검색 메소드 추가 - AccommodationRepository 에 입력된 검색 조건에 따라 검색을 수행하는 메소드를 추가하였습니다. * Feat: AccommodationService search() 메소드 로직 수정 - 기존의 자바 코드 기반으로 검색을 수행하던 로직을, DB의 쿼리 기반으로 로직을 수행하게 변경하였습니다. * Refactor: Accommodation 코드 포맷팅 수정 - 실수로 대괄호가 하나 들어가 있어 삭제하였습니다. * [Team22][FE]1주차 금요일 pr (street62#98) * Refactor: PR 리뷰를 바탕으로 코드 수정 - 모달창 경로 수정 - styled-components와 mui 구분 - 그 외는 리뷰 참고 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * [FE] result페이지 UI 구현 (street62#33) * Design: 기존 스타일 수정 - 라우터에 따른 스타일 변경 추가 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Design: Result 페이지 UI 구현 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Rename: Hotel.tsx 파일 경로 이동 * Design: result 페이지의 숙소 리스트 컴포넌트와 일부 컴포넌트의 스타일 변경 * Design: Hotel 컴포넌트와 관련된 스켈레톤 UI 생성 * Refactoer: 서치바 리팩토링 - 각 컴포넌트를 공통된 스타일 컴포넌트로 변경 - 일부 태그 변경 - hover 이펙트 추가 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Chore: dependencies 관련 수정 (street62#40) - devDependencies에 설치된 스타일 라이브러리를 dependencies로 재설치함 Co-authored-by: jm <jm.dev.study@gmail.com> Co-authored-by: herrakam <herrakam@users.noreply.github.com> Co-authored-by: 도비 <85747667+JiminKim-dev@users.noreply.github.com> * Refactor: 엔티티 애노테이션 순서 통일 (street62#42) Lombok 애노테이션부터 생성자 - getter 순서로. 이후에는 JPA @entity 애노테이션으로 이어지도록 수정. Co-authored-by: Senglee Choi <tmdfl1027@gmail.com> * Dto, 레포지토리, 엔티티 관련 수정 (street62#46) * Fix: Accommodation mappedBy 수정, 최대인원 합계 메서드 추가 - mappedBy 속성에 reservation 엔티티에서 해당하는 필드인 accommodation으로 값 수정. - 검색 response DTO에 사용할 최대인원 합계 반환 메서드 작성. * Refactor: Accommodation 필드 변경으로 인한 Builder 호출 메서드 변경 Accommodation 필드 구성 변경에 맞는 값을 반환하는 메서드를 빌더에서 호출하도록 변경. * Refactor: SearchQueryRequestDto 수정 - 날짜 매핑을 위해 @DateTimeFormat 을 SearchQueryRequestDto 에 추가하였습니다. * Refactor: User 엔티티 클래스 이름 수정 - User 가 MySQL 예약어이기에, 엔티티 이름을 Member 로 변경하였습니다. * Refactor: 검색 조회 쿼리 수정 - 기존의 검색 조회 쿼리가 작동하지 않아, 수정하였습니다. Co-authored-by: bugpigg <bugpigg@gmail.com> * 숙소 예약 페이지 GET API 구현 (street62#51) * feat: AccommodationController에 숙소 예약 페이지 반환 메소드 추가 - AccommodationController에 숙소 예약 페이지 반환 메소드를 추가하였습니다. * Feat: AccommodationService에 숙소 예약 페이지 반환 메소드 추가 - AccommodationService에 숙소 예약 페이지 반환 메소드를 추가하였습니다. Accommodation 엔티티의 id 컬럼을 활용하여 DB에서 해당 숙소의 정보를 조회하고 관련 DTO를 생성하여 컨트롤러 레이어로 반환합니다. * Feat: ReserveFormResponseDto에 정적 팩토리 메소드 추가 - ReserveFormResponseDto에 Accommodation 객체를 받아 ReserveFormResponseDto 객체를 생성하는 정적 팩토리 메소드를 추가하였습니다. * [BE] 숙소 예약 페이지 POST API 구현 (street62#54) * Feat: MemberRepository 클래스 추가 Member 엔티티의 DB 접근 계층인 MemberRepository 클래스를 추가했습니다. * Feat: ReservationRepository 추가 Reservation 엔티티의 DB 접근 계층인 ReservationRepository 클래스를 추가. * Refactor: ReserveRequestDto 필요 필드 추가 Reservation을 생성하는 데에 필요한 필드들을 추가로 선언함. * Refactor: Reservation builder 추가 * Feat: 예약 POST API 구현 컨트롤러에서 AccommodationService 계층의 generateNewReservation() 메서드를 호출해 새 예약을 생성해 DB에 저장하도록 함. * [BE] 숙소 예약 페이지 GET API 수정 (street62#55) * Refactor: DiscountPolicy id 필드 수정 및 Getter 추가 - 기존 `long`타입이던 id 필드를 `Long` 타입으로 수정하였습니다. - DiscountPolicy 에 Getter 롬복 어노테이션을 추가하였습니다. * Feat: ReserveFormResponseDto 에 discountRate 필드 추가 - ReserveFormResponseDto 에 discountRate 필드를 추가하였습니다. * [BE] 코드 리팩토링 (street62#57) * Feat: 예외처리 핸들러 클래스 및 사용자 정의 예외 추가 - 예외처리 핸들러 클래스 및 사용자 정의 예외를 추가하였습니다. * Feat: 예외 Response DTO 클래스 추가 - 예외 발생시 반환되는 ErrorResponse DTO 클래스를 추가하였습니다. * Feat: Member, Accommodation 엔티티 검색 실패 예외 추가 및 서비스 클래스에 반영 - MemberNotFoundException, AccommodationNotFoundException 클래스를 추가하였습니다. - 추가한 예외들을 AccommodationService 클래스에 반영하였습니다. * Refactor: 도메인 클래스 관련 폴더 구조 변경 - 도메인 클래스에 대하여 해당 도메인 클래스의 이름을 가지는 패키지 아래 위치하도록 변경하였습니다. * Refactor: 서비스, 컨트롤러 메서드명 변경 서비스, 컨트롤러 계층의 메서드명을 명료하게 수정하였습니다. Co-authored-by: bugpigg <bugpigg@gmail.com> * [BE] 초기 데이터 dml.sql, ddl.sql 작성 및 스프링 프로필 설정 (street62#58) * Feat: 스프링 프로필 설정파일 및 로그 설정 파일 추가 - 스프링 프로필 설정 파일인 application.yml 파일을 추가하였습니다. - 로그 설정 파일인 logback-spring.xml 파일을 추가하였습니다. * Feat: dml.sql 추가 - 초기 테스트용 더미 데이터 추가를 위한 dml.sql 파일을 추가하였습니다. * AccommodationController의 RegisterReservation() 반환타입 변경 및 ReserveRequestDto의 날짜 매핑 어노테이션 추가 (street62#60) * Fix: ReserveRequestDto의 날짜 매핑 어노테이션 추가 - 기존 날짜 매핑 어노테이션이 빠져 있어, ReserveRequestDto의 날짜 매핑 어노테이션을 추가하였습니다. * Feat: AccommodationController의 RegisterReservation() 반환타입 변경 - 기존 HttpStatus를 반환하던 RegisterReservation() 반환타입을 ResponseEntity 로 수정하였습니다. * Fix: 숙소 검색 쿼리 수정 (street62#63) - 숙소 검색 쿼리가 구현한대로 동작하지 않아, 수정하였습니다. * [FE] [Team-22/도비 & JS] 2주차 첫번째 PR (codesquad-members-2022#154) * Refactor: PR 리뷰를 바탕으로 코드 수정 - 모달창 경로 수정 - styled-components와 mui 구분 - 그 외는 리뷰 참고 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * [FE] result페이지 UI 구현 (street62#33) * Design: 기존 스타일 수정 - 라우터에 따른 스타일 변경 추가 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Design: Result 페이지 UI 구현 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Rename: Hotel.tsx 파일 경로 이동 * Design: result 페이지의 숙소 리스트 컴포넌트와 일부 컴포넌트의 스타일 변경 * Design: Hotel 컴포넌트와 관련된 스켈레톤 UI 생성 * Refactoer: 서치바 리팩토링 - 각 컴포넌트를 공통된 스타일 컴포넌트로 변경 - 일부 태그 변경 - hover 이펙트 추가 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * Fix: PR 리뷰 반영 - InputState 네이밍 변경 - position의 타입 지정은 추후 수정하기 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * 가격, 요금, 인원에 대한 Context 작성 (street62#47) * Feat : PeriodContext 구현, PeriodModal과 연결중 * Feat: PriceContext와 PersonnelContext 구현 * Feat: Context 적용 * Design: Modal display 변경 Co-authored-by: SeungHyun <fm10033@gmail.com> * Feat: 각 영역 클릭시 모달 여닫는 기는 구현 * Feat : periodContext 로직 최신 날짜 반영으로 변경, modal에 적용 * Fix: 각 영역의 ID값으로 clickModal 함수를 실행하도록 수정 * Fix: 중복되는 함수를 Props로 받도록 수정 * Design : 달력 기본 레이아웃 구현 * Feat: 각 영역 클릭시 강조되는 효과 추가 * Feat : 화살표 클릭시 달 바뀌는 기능 구현 * Feat: 모달이 활성화되면 각 영역의 취소버튼이 보이도록 추가 * Feat : 달력 요일 구현 * Refact : 화살표 클릭시 달 변경 1월~12월 까지만 동작하게 변경 * Refactor: 재사용 가능한 컴포넌트로 분리 * Fix: id 속성을 dataset 으로 변경 * Refactor: ModalProvider 내부 함수를 useModal로 분리 * Style: 불필요한 주석 제거 * Fix: InputButton에 defaultProps 설정 * Style: 의미없는 매직넘버 제거 * Feat: 유저 메뉴와 관련된 컴포넌트 분리 * Refactor: pathname 처리 로직 수정 * Feat: 검색버튼 클릭시 result 페이지로 이동하도록 구현 Co-authored-by: herrakam <herrakam@users.noreply.github.com> Co-authored-by: SeungHyun <fm10033@gmail.com> * [Team22][FE] 도비 & JS 2주차 금 PR (codesquad-members-2022#219) * Fix: PR 리뷰 반영 - InputState 네이밍 변경 - position의 타입 지정은 추후 수정하기 Co-authored-by: herrakam <herrakam@users.noreply.github.com> * 가격, 요금, 인원에 대한 Context 작성 (street62#47) * Feat : PeriodContext 구현, PeriodModal과 연결중 * Feat: PriceContext와 PersonnelContext 구현 * Feat: Context 적용 * Design: Modal display 변경 Co-authored-by: SeungHyun <fm10033@gmail.com> * Design : 달력 기본 레이아웃 구현 * Feat : 화살표 클릭시 달 바뀌는 기능 구현 * Feat : 달력 요일 구현 * Refact : 화살표 클릭시 달 변경 1월~12월 까지만 동작하게 변경 * 리뷰 적용중 * Fix: 클릭 이벤트 콜백 함수 타입으로 변경 * Fix: context에서 가져오는 state의 이름 변경 * 컨플해결중 * Refact : 컨플릭트 해결 * Refact : 피드백 적용 * Refact : periodContext 구조 변경, 월 변경 기능 수정 month만 전달하던 구조에서 Date 객체 전체를 전달하는 구조로 변경 해가 넘어 월 변경이 구현되게 수정 * Refact: 연도 변화 오류 해결 * Feat : 다음 달 달력 구현 * Feat : 달력 시작 요일 반영 * Feat : 클릭 이벤트 구현중 * [FE] 금액 차트 구현 (street62#72) * Rename: 파일 이름 변경 및 폴더 이동 * Feat: Price와 관련된 mock데이터 생성 및 모달에 적용 - 천단위 숫자 구분을 위해 toLocaleString 헬퍼함수 구현 * Feat: 차트가 들어갈 캔버스 생성 * Feat: 곡선 차트 구현 * Style: 식을 변수로 분리, 변수 이름 변경 * Style: 객체의 키값 이름 변경 * Design: 범위 슬라이더 UI 구현 (미완성) * Move: 파일 정리 * Refactor: 데이터의 범위 간격을 구하는 함수 수정 Co-authored-by: herrakam <herrakam@users.noreply.github.com> Co-authored-by: SeungHyun <fm10033@gmail.com> Co-authored-by: Senglee Choi <tmdfl1027@gmail.com> Co-authored-by: SeungHyun <fm10033@gmail.com> Co-authored-by: jm <jm.dev.study@gmail.com> Co-authored-by: 도비 <85747667+JiminKim-dev@users.noreply.github.com> Co-authored-by: herrakam <herrakam@users.noreply.github.com> Co-authored-by: HYUNJUN SON <55608425+guswns1659@users.noreply.github.com> Co-authored-by: herrakam <54533561+herrakam@users.noreply.github.com>
torch-ray
pushed a commit
that referenced
this pull request
Jun 10, 2022
…r_implementation Feature ios/#95 network manger implementation
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주차 금요일 pr 요청드립니다!
수요일에 자세한 PR 리뷰 감사합니다!!
구현한 것
질문