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

[Others] 예정 진행중 토너먼트 데이터 fetching 리팩토링 #1024 #1218

Conversation

Clearsu
Copy link
Collaborator

@Clearsu Clearsu commented Dec 27, 2023

📌 개요

  • useBeforeLiveTournamentData 커스텀훅에서 예정 및 진행중인 토너먼트 배열을 분리하여 받을 수 있도록 수정하여 메인 페이지 및 토너먼트 페이지에서 재사용할 수 있도록 리팩토링하였습니다.
  • 토너먼트 페이지의 예정 및 진행중인 토너먼트 데이터 요청 로직을 useBeforeLiveTournamentData 훅으로 대체하였습니다.
  • 사용하지 않는 TournamentPreviewItem 컴포넌트를 삭제하고, 디렉토리 구조를 변경하였습니다.

@Clearsu Clearsu self-assigned this Dec 27, 2023
@Clearsu Clearsu added user 일반 사용자가 사용하는 서비스에서 발생한 issue refactor 유지보수와 잠재적 오류를 방지하기 위한 refactor issue labels Dec 27, 2023
Copy link
Collaborator

@greatSweetMango greatSweetMango left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT!

@Clearsu Clearsu merged commit ea3c625 into main Jan 1, 2024
1 check passed
@Clearsu Clearsu deleted the others/예정-진행중-토너먼트-데이터-fetching-리팩토링-#1024 branch January 1, 2024 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 유지보수와 잠재적 오류를 방지하기 위한 refactor issue user 일반 사용자가 사용하는 서비스에서 발생한 issue
Projects
Status: in-review
Development

Successfully merging this pull request may close these issues.

[Others] 예정 & 진행중 토너먼트 데이터 fetching 리팩토링
2 participants