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.

Sign up for GitHub

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

Jump to bottom

[FE] 북마크 및 전체보기 페이지 API 로직 수정 및 React-Query 적용 #642

Closed
3 tasks done
semnil5202 opened this issue Jan 10, 2024 · 0 comments
Closed
3 tasks done

[FE] 북마크 및 전체보기 페이지 API 로직 수정 및 React-Query 적용 #642

semnil5202 opened this issue Jan 10, 2024 · 0 comments
Assignees
@semnil5202
Labels
FE 프론트엔드 관련 이슈 refactor 리팩토링 관련 이슈

Comments

@semnil5202
Copy link
Collaborator

semnil5202 commented Jan 10, 2024 •
edited
Loading

🛠️ 작업 대상

  • 북마크 페이지
  • 각 항목(인기, 새로운, 모두)별 토픽 전체보기 페이지
  • 위 페이지에서 사용되는 컴포넌트

✅ 작업 내용

  • 페이지에서 사용되는 컴포넌트 검토
  • API 로직 수정 Axios 적용
  • API 요청 훅 React-Query 도입

📎 참고 자료

No response

⏰ 추정 시간

24.01.13 토요일까지
@semnil5202 semnil5202 added the refactor 리팩토링 관련 이슈 label Jan 10, 2024
@semnil5202 semnil5202 self-assigned this Jan 10, 2024
@semnil5202 semnil5202 added the FE 프론트엔드 관련 이슈 label Jan 10, 2024
@semnil5202 semnil5202 mentioned this issue Jan 14, 2024
Merged
4 tasks
jiwonh423 added a commit that referenced this issue Jan 21, 2024
@jiwonh423
Merge branch 'develop-FE' into refactor/#642
1e1a7a3
jiwonh423 added a commit that referenced this issue Jan 21, 2024
@semnil5202 @jiwonh423
[FE] Refactor/#642: 북마크 및 전체보기 페이지 API 로직 수정 및 React-Query 적용 (#644)
f7d8897 
* feat: Axios http 정의 및 get 메서드 구현

* refactor: Bookmarks 페이지 TopicCardList 분리 코드 합병

TopicCardList는 재사용을 위해 만들어진 코드이나 Axios, React Query 를 적용하면서 사용하기 애매해졌다.
이에 따라 TopicCardList를 제거하고 각 페이지에서 그 책임을 이어받는다.

* design: Skeleton 컴포넌트 스타일 변경 및 적용

* refactor: SkeletonBox 공통 컴포넌트 구현 및 convertCSS 유틸 함수 구현

* refactor: Skeleton 컴포넌트 Bookmark 페이지에 적용

* refactor: http api 수정 및 useGetBookmark isLoading 상태 사용

* refactor: SeeAllNearTopics 페이지 TopicCardList 분리 및 코드 합병

* refactor: API 로직 반환값 타입 지정

* refactor: Bookmark 스켈레톤 수정

* refactor: SeeAllLatestTopics 페이지 TopicCardList 분리 및 코드 합병

* refactor: SeeAllPopularTopics 페이지 TopicCardList 분리 및 코드 합병

* refactor: AllTopics Query key 수정 및 시맨틱 태그 수정

* rename: API 명세와 페이지 이름 통일화

* refactor: 전체보기 페이지 명칭 수정 router 적용

* feat: 리프레쉬 토큰 요청 기능 추가

운영서버에 머지해야 확인가능할 것 같습니다.
기존 로직을 사용하는 운영서버와 request 값은 동일한데 cookie와 도메인 설정 문제로 실패하는 것으로 확인됩니다.

* feat: query default option 설정

* 마운트 시 리페칭 해제
* 윈도우 포커스 시 리페칭 해제
* 일정 주기로 리페칭 해제
* 받아온 데이터 stale 시 리페칭 해제

* refactor: useSuspenseQuery 를 통한 선언적으로 로딩상태 처리

* fix: token 없을 때 Authorization 빈 객체로 세팅하여 비로그인 오류 해결

* refactor: withCredentials 옵션 잠시 보류

* refactor: 01.17 회의를 통한 변경

* TopicCardList 컴포넌트를 이전처럼 활용하도록 한다. 전체보기 및 즐겨찾기는 거의 동일한 형태이며 중복코드가 다량 발생하여 위와 같이 수정한다.
* url을 넘겨받음에 따라서 리액트 쿼리 훅, API 요청 로직 또한 하나의 훅으로 재사용한다.

---------

Co-authored-by: afds4567 <[email protected]>
semnil5202 pushed a commit that referenced this issue Feb 5, 2024
@jiwonh423
[FE] refactor after merge #643, #642 (#649)
d77d381 
* Refactor : Remove unused code and update import path

* Refactor : Update default topic and profile images

* Refactor :  Profile component and update routingHandlers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@semnil5202 semnil5202

Labels
FE 프론트엔드 관련 이슈 refactor 리팩토링 관련 이슈
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@semnil5202