-
Notifications
You must be signed in to change notification settings - Fork 4
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
리팩토링 #95
Labels
Comments
2hoyeong
added a commit
that referenced
this issue
Dec 6, 2020
구글 맵 API를 여러 곳에서 호출하고 있어서 컨텍스트로 한곳에서 관리할 수 있도록 컨텍스트를 생성했습니다.
2hoyeong
added a commit
that referenced
this issue
Dec 6, 2020
구글 맵 API 사용하는 곳 구현된 context 사용할 수 있도록 수정했습니다.
2hoyeong
added a commit
that referenced
this issue
Dec 6, 2020
구글 맵 API의 directionRenderer 타입을 설정했습니다.
2hoyeong
added a commit
that referenced
this issue
Dec 6, 2020
1초마다 현재 위치를 받아와 스토어에 업데이트 하는 것을 맵 컴포넌트에서 맵 컨테이너로 이동했습니다. useSelector에 shallowEqual을 이용해 location의 값이 변하지 않았다면 재렌더링 되지 않도록 했습니다.
2hoyeong
added a commit
that referenced
this issue
Dec 6, 2020
맵 컴포넌트를 처음 렌더링 했을때 현재 위치의 가장 가까운 가게를 자동으로 시작지점으로 지정하는 함수를 MapContainer에서 InputPlaces로 이동시켰습니다.
GrasshopperBears
added a commit
that referenced
this issue
Dec 7, 2020
GrasshopperBears
added a commit
that referenced
this issue
Dec 7, 2020
- dispatch 사용 함수의 useCallback 사용 보류
GrasshopperBears
added a commit
that referenced
this issue
Dec 10, 2020
- 선택되지 않은 곳에 메세지 출력 - 선택되지 않은 placeholder 색상 변경
GrasshopperBears
added a commit
that referenced
this issue
Dec 11, 2020
GrasshopperBears
added a commit
that referenced
this issue
Dec 11, 2020
- mutation 폴더에 들어있던 resolver들을 역할에 맞게 분리
GrasshopperBears
added a commit
that referenced
this issue
Dec 11, 2020
- requestingUser 정보가 누적되는 버그가 생겨 DB에서 제거하도록 변경
GrasshopperBears
added a commit
that referenced
this issue
Dec 11, 2020
- try/catch로 에러 처리 - 에러 메세지 분리
GrasshopperBears
added a commit
that referenced
this issue
Dec 11, 2020
GrasshopperBears
added a commit
that referenced
this issue
Dec 11, 2020
GrasshopperBears
added a commit
that referenced
this issue
Dec 11, 2020
GrasshopperBears
added a commit
that referenced
this issue
Dec 11, 2020
- graphql 스키마에 추가 - 변경된 스키마에 맞춰 query 수정
GrasshopperBears
added a commit
that referenced
this issue
Dec 11, 2020
- 스키마에 HistoryInfo 타입 추가 - 이에 따른 query 수정
Merged
ahnsoheee
added a commit
that referenced
this issue
Dec 12, 2020
ahnsoheee
added a commit
that referenced
this issue
Dec 12, 2020
ahnsoheee
added a commit
that referenced
this issue
Dec 12, 2020
ahnsoheee
added a commit
that referenced
this issue
Dec 12, 2020
- 드라이버/유저 로그인 뮤테이션을 하나로 통합 - type(user/driver)에 따라 해당 스키마에서 아이디와 비밀번호를 조회
ahnsoheee
added a commit
that referenced
this issue
Dec 12, 2020
- 기존 드라이버/유저 로그인 삭제 - 통합된 signin 뮤테이션 추가
Merged
ahnsoheee
added a commit
that referenced
this issue
Dec 13, 2020
- 드라이버/유저 로그인 뮤테이션을 하나로 통합 - type(user/driver)에 따라 해당 스키마에서 아이디와 비밀번호를 조회
GrasshopperBears
added a commit
that referenced
this issue
Dec 14, 2020
- requestingUser에 해당 유저가 요청한 document는 모두 제거하도록 변경
GrasshopperBears
added a commit
that referenced
this issue
Dec 14, 2020
- 드라이버가 내역 조회 버튼을 클릭할 경우 /user/history로 이동하는 버그 수정
GrasshopperBears
added a commit
that referenced
this issue
Dec 14, 2020
- handleSubmit을 useCallback으로 변경 - 번호와 이름을 비구조화 할당으로 변경
GrasshopperBears
added a commit
that referenced
this issue
Dec 14, 2020
- onClickHanlder를 MatchingCancelButton으로 이동
GrasshopperBears
added a commit
that referenced
this issue
Dec 14, 2020
- 유저 매칭 페이지에서 불필요한 비동기 처리 제거 - 예상 요금에 toLocaleString 적용
GrasshopperBears
added a commit
that referenced
this issue
Dec 14, 2020
- pathPoint를 Request로 바꿔주는 함수 분리 - request를 state에서 제거
GrasshopperBears
added a commit
that referenced
this issue
Dec 14, 2020
- startTime을 state 대신 ref로 상태 관리
GrasshopperBears
added a commit
that referenced
this issue
Dec 14, 2020
- taxiInfo가 변경될 일이 없기 때문에 useMemo로 래핑
Merged
GrasshopperBears
added a commit
that referenced
this issue
Dec 14, 2020
GrasshopperBears
added a commit
that referenced
this issue
Dec 16, 2020
- 매칭된 경우에도 요청을 계속 전송하는 문제 - 매칭된 경우 clearInterval을 하도록 변경
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
개요 🔍
완료 조건 ✅
참조 📎
The text was updated successfully, but these errors were encountered: