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

[FE] 토픽 단일 조회 페이지 레이어 분리 #655

Closed
3 tasks done
semnil5202 opened this issue Feb 29, 2024 · 0 comments
Closed
3 tasks done

[FE] 토픽 단일 조회 페이지 레이어 분리 #655

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

Comments

@semnil5202
Copy link
Collaborator

semnil5202 commented Feb 29, 2024

🛠️ 작업 대상

  • SelectedTopic 페이지 및 그와 관련된 로직들

✅ 작업 내용

  • React Query 적용
  • 비즈니스 레이어 (custom hook, service) 적용으로 레이어 분리
  • 그외 자잘한 오류 수정 (SSOT 등)

📎 참고 자료

No response

⏰ 추정 시간

3월 첫째 주까지

@semnil5202 semnil5202 added FE 프론트엔드 관련 이슈 refactor 리팩토링 관련 이슈 labels Feb 29, 2024
@semnil5202 semnil5202 self-assigned this Feb 29, 2024
@semnil5202 semnil5202 changed the title 토픽 단일 조회 페이지 레이어 분리 [FE] 토픽 단일 조회 페이지 레이어 분리 Feb 29, 2024
semnil5202 added a commit that referenced this issue May 4, 2024
* rename: 페이지 단위 코로케이션 적용

* refactor: 불필요한 변수 제거

* refactor: topicDetail 요청 로직 Query로 분리

* refactor: 클러스터링 로직 훅으로 분리 및 재사용

추후 모아보기 페이지에서 재사용할 예정이다.

* refactor: 사용자 지도 상호작용 이벤트 핸들링 및 상호작용 후 동작 로직 커스텀 훅으로 분리

* chore: prettier printWidth 100으로 설정

훅의 인자, 반환값의 네이밍 길이가 길어짐에 따라 위와 같이 변경한다.

* refactor: 더블 사이드바 및 쿼리파라미터 세팅 로직 커스텀 훅으로 분리

추후 모아보기 페이지에서 사용할 수 있도록 한다.

* refactor: useTags init 로직 useTags 훅 내부로 이동

모든 페이지마다 tags를 초기화해줘야하는 로직이 불필요하게 반복된다. 또한 휴먼에러가 발생할 가능성이 높은 부분으로 판단되어 useTags의 props로 받아 초기화가 불필요한 페이지는 명시적으로 초기화 하지 않도록 한다.

* fix: tags 초기화 기능 조건 로직 반대로 지정한 오류 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈 refactor 리팩토링 관련 이슈
Projects
Status: Done
Development

No branches or pull requests

1 participant