Refactor/api #162

sungwoo-shin · 2022-08-19T08:51:29Z

📝 PR Summary

반드시 Refactor/norm #160 리뷰 이후 본 PR을 리뷰해주세요!!
백 API response 변경에 대응
- [REFACTOR] 백엔드 리팩토링으로 인한 API 수정사항 대응 #161 의 TODO중 '규진 작업부탁해요'로 표시한 부분은 추가 작업 부탁드립니다.
멘토링 시 지적받은 개선사항 리팩토링
React Query 도입과 migration 예정으로 인해 다음의 API 관련 리팩토링은 진행하지 않습니다. (migration 이후 다음 사항중 필요여부 검토 후 리팩토링 진행하겠습니다.)
- hasAlreadyBeenFetched 함수 통합
- slice 파일에서 thunk 분리
- endpoint constant로 별도 파일로 관리

🌲 Working Branch

refactor/API

🌲 TODOs

Related Issues

#161

- 블록 스코프에서는 함수 선언식을 사용하지 않는다. - 함수 내에서 반환은 한 번만 한다.

- anti pattern 해소 - moment.js -> day.js

This reverts commit c12a1b9.

- 배경 아래로 깔리는 animation 적용 시 CSS 작업에 시간을 과도하게 사용할것 같아 - color에만 transition을 적용합니다. - 부모와 자녀가 동일한 transition을 경험합니다.

- thunk에서 error 발생 시 rejected case 이동 시 action.payload undefined 상태임 - rejectWithValue 사용해서 error를 전역상태로 추가로 관리해야함 - React Query migration 시 에러처리 로직 추가하고, 알파버전 릴리즈 시에는 에러처리 없이 가는게 나을것 같음

sungwoo-shin added 26 commits August 15, 2022 20:10

fix: ESLint eqeqeq rule 적용

983e8c2

refactor: SuggetBadeg -> ProposalBadge 네이밍 변경

b4ca245

fix: == -> ===

0a67fb7

feat: disable shouldCloseOnOverlayClick

9770d78

refactor: anti pattern 개선

d018376

- 블록 스코프에서는 함수 선언식을 사용하지 않는다. - 함수 내에서 반환은 한 번만 한다.

feat: HomeTemplate mount 시 animation 삭제

16beccb

refactor: /utils/get 개선

e39079e

- anti pattern 해소 - moment.js -> day.js

refactor: 하나의 함수에서는 하나의 return만 하도록 개선

0f941f4

chore: 복잡한 컴포넌트에 대해 주석 작성

f276c7a

pref: 온보딩 input 입력 시 useCallback 적용

c12a1b9

Merge branch 'dev' into refactor/norm

69af645

fix: == -> ===

8e8acbd

refactor: CSS 코드 간소화

3af96a2

chore: 함수명 변경

5e0f82b

Revert "pref: 온보딩 input 입력 시 useCallback 적용"

d06987b

This reverts commit c12a1b9.

규진 코드에서 개선할 수 있는 부분 마킹

358caab

refactor: 돈길 리스트 쿼리파람 pending 유지, accept -> walking

640b8d9

fix: conflict

10a0689

refactor: challengeStatus ENUM 적용

9c04180

refactor: level: 0 -> level: -4

aa239f6

refactor: Modals 코드 가독성 개선

5dbe880

refactor: HomeTemplate 가독성 개선

cda4551

refactor: refineBirthday 재사용성 개선

17d99c2

refactor: convertTimeStampToYYYYMMDD 재사용성 개선

1125302

refactor: isSelectedKid 조건 유지보수성 개선, map method 사용 시 key 최상단 위치

f819350

refactor: RegisterBirthday boolean 변수명 개선, 불필요한 logical test 삭제

741aa62

sungwoo-shin requested a review from 9yujin August 19, 2022 08:51

sungwoo-shin added 2 commits August 19, 2022 18:00

9yujin approved these changes Aug 19, 2022

View reviewed changes

9yujin merged commit 0f5a923 into dev Aug 19, 2022

9yujin deleted the refactor/API branch August 19, 2022 16:10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Refactor/api #162

Refactor/api #162

sungwoo-shin commented Aug 19, 2022 •

edited

Loading

Refactor/api #162

Refactor/api #162

Conversation

sungwoo-shin commented Aug 19, 2022 • edited Loading

📝 PR Summary

🌲 Working Branch

🌲 TODOs

Related Issues

sungwoo-shin commented Aug 19, 2022 •

edited

Loading