Skip to content
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

카테고리 리덕스 세팅 및 페이지에 적용 #124

Merged
merged 59 commits into from
Mar 11, 2021

Conversation

anxiubin
Copy link
Member

@anxiubin anxiubin commented Feb 16, 2021

카테고리 컴포넌트 재구성

  • 기존에 있던 CategoryTabDrawer를 분해하여 CategoryList 하위에 컴포넌트를 만들고 재구성
    => CategoryButtonGroup, CategoryHeader, CategoryListItem 생성

카테고리 api 모듈 생성 및 개선

  • Redux store에 카테고리 리스트 데이터 저장
  • 공통으로 사용할 프레임에 맞춰서 모듈 개선

LinkDropZone 컴포넌트 handleDropOnCardArea 로직 개선

  • linkCreateThunk 실행 이후에 필요한 로직들 추가

코드 리뷰 적용 (21/03/04 기준)

@yjkwon07
Copy link
Contributor

yjkwon07 commented Feb 16, 2021

src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/CategoryListItem/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/index.jsx Outdated Show resolved Hide resolved
src/pages/category/CategoryList/index.jsx Outdated Show resolved Hide resolved
@yjkwon07 yjkwon07 added enhancement New feature or request Front-Component Front Component labels Feb 17, 2021
anxiubin and others added 28 commits March 1, 2021 18:41
- reducer state는 아직 사용하지 않아서 정의 안 함
- Header 부분 탭 열기/ 삭제, Search 구현 해야 함
- 기본으로 bottom 영역에 생기도록 스타일 수정
- redux searchFilter 관리
- SearchBar 공통 컴포넌트로 이동
- SearchButton ToggleButton 추가
- RefreshButton 추가
- TabButtonGroup 추가
- reload: 검색어 포함 다시 로드, refresh: 초기 값으로 다시 로드(변수명 정의 변경)
- detact로 reload 감지를 할지 결정
- linkRead 할 때, categoryId, 검색 파라미터가 한번에 바뀌지 않음
  - 네트워크 불필요한 요청들이 나가게 됨
  - takeLatest => debounce로 파라미터 변경 시간을 고려하여 조절
  - debounce 1000 => 100으로 변경
- multiline single noWrap 형식처럼 ...  대체를 css로 변경함
- section, main으로 layout 설정
- useCallback 추가 및 필요없는 import, 함수 사용 제거
@yjkwon07 yjkwon07 merged commit 8bcc948 into develop Mar 11, 2021
@yjkwon07 yjkwon07 deleted the feature/redux-setting branch March 16, 2021 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Front-Component Front Component
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants