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

[Others] 토너먼트 브라켓 및 스와이퍼 로딩 컴포넌트 적용 #1203 #1227

Merged
merged 39 commits into from
Jan 4, 2024
Merged

[Others] 토너먼트 브라켓 및 스와이퍼 로딩 컴포넌트 적용 #1203 #1227

merged 39 commits into from
Jan 4, 2024

Conversation

Clearsu
Copy link
Collaborator

@Clearsu Clearsu commented Jan 4, 2024

📌 개요

  • 유저 토너먼트 브라켓 및 유저 토너먼트 전적 페이지의 스와이퍼에 로딩 UI를 적용하였습니다.
  • 전
    전적페이지렌더링이슈
  • 후
    리그변경간로딩컴포넌트

💻 작업사항

  • WinnerSwiper 내부의 토너먼트 데이터 요청 로직을 상위 컴포넌트로 이동하여 UI 컴포넌트의 역할을 명확히 하였습니다.
  • 리그 타입 변경 시(쿼리 키) 쿼리가 초기화되도록 하여 이전 데이터가 남아 보이지 않도록 하였습니다.
    • useInfiniteQuery의 keepPreviousData 옵션이 활성화되어있을 경우 초기 로드 시에만 isLoading이 true가 되어 리그 타입 변경 시 데이터 요청 간 로딩 컴포넌트를 표시할 수 없습니다.
  • Tournament 페이지 컴포넌트를 리팩터링 하였습니다.
    • 예정 및 진행중 토너먼트 데이터를 패칭하는 useBeforeLiveTournamentData 훅, 내부에서 진행중인 토너먼트 게임 데이터를 요청하고 브라켓으로 보여주는 UserTournamentBracket 컴포넌트로 중복 로직을 대체하였습니다.
  • 현재 사용하지 않는 useContainerSize 훅을 삭제하였습니다.

Clearsu added 30 commits December 20, 2023 12:47
@Clearsu
[Fix] 진행중인 토너먼트 없을 시 '0' 렌더링 수정 #1173
e4cf748
@Clearsu
[Chore] 파일 경로 변경 #1173
e6a3cb9
@Clearsu
[Feat] boucing dots UI 컴포넌트 추가 #1173
8cb65e1
@Clearsu
[Feat] bouncing dots 컴포넌트 추가 #1173
2e7be20
@Clearsu
[Style] 패자 어두운 오버레이 적용 #1173
45fbf99
@Clearsu
Merge branch 'main' of github.com:42organization/42gg.client into oth…
f627514 
…ers/토너먼트-브라켓-승자-패자-구분-ui-#1173
@Clearsu
[Refactor] 사용하지 않는 Import 제거
5ffde25
@Clearsu
[Feat] 경기 상태에 따른 bouncing dots UI 표시 #1173
b85b359
@Clearsu
[Refactor] before, live 데이터를 분리하여 반환하도록 수정 #1204
a67dca4
@Clearsu
[Refactor] 로딩컴포넌트 대체 #1203
2bf417c
@Clearsu
[Refactor] loading spinner 컴포넌트 분리 및 fallbackText 추가 #1203
28a69e6
@Clearsu
[Refactor] loading spinner 컴포넌트 분리 #1203
d30758c
@Clearsu
[Chore] 컴포넌트 경로 변경 #1203
9cdede9
@Clearsu
[WIP] 리팩토링 중 #1203
b483430
@Clearsu
[Docs] 주석 제거 #1203
9c03732
@Clearsu
[Chore] 경로 변경 #1203
93be361
@Clearsu
[Refactor] loading spinner #1203
2fe673f
@Clearsu
Merge branch 'main' of github.com:42organization/42gg.client into 120…
d6b432f 
…4-others-예정-진행중-토너먼트-데이터-fetching-리팩토링
@Clearsu
[Chore] 디렉토리 삭제 #1204
4b68c52
@Clearsu
[Chore] 미사용 컴포넌트 제거 #1204
f85847d
@Clearsu
[Refactor] 의존성배열 수정, 참가 상태 문자열 수정 #1204
7e6e925
@Clearsu
[Refactor] 리턴 타입 명시 #1204
d6fcab8
@Clearsu
[Refactor] 예정&진행중인 토너먼트 data fetching 커스텀훅 적용 #1204
9f7eb02
@Clearsu
[Style] noTournamentText 색상 및 폰트 사이즈 조정 #1204
f089a9d
@Clearsu
[Style] gap 및 폰트 사이즈 수정 #1204
cca7478
@Clearsu
Merge branch 'others/예정-진행중-토너먼트-데이터-fetching-리팩토링-#1024' of github.c…
7de7da0 
…om:42organization/42gg.client into others/토너먼트-브라켓-및-스와이퍼-로딩-컴포넌트-적용-#1203
@Clearsu
[Refactor] 데이터 요청 로직 분리 #1203
55c436e
@Clearsu
[Refactor] 사용하지 않는 스타일 삭제 #1203
33c5605
@Clearsu
[Style] fallback 문구 수정 #1203
74ccebf
@Clearsu
[Refactor] stale time 프롭 추가 및 fallback text 제거 #1203
6ef6406
@Clearsu Clearsu added user 일반 사용자가 사용하는 서비스에서 발생한 issue refactor 유지보수와 잠재적 오류를 방지하기 위한 refactor issue labels Jan 4, 2024
@Clearsu Clearsu self-assigned this Jan 4, 2024
@Clearsu Clearsu linked an issue Jan 4, 2024 that may be closed by this pull request
[Others] 토너먼트 브라켓 및 스와이퍼 로딩 컴포넌트 적용 #1203
Closed
joonho0410
joonho0410 approved these changes Jan 4, 2024
View reviewed changes
Copy link
Collaborator

@joonho0410 joonho0410 left a comment

Choose a reason for hiding this comment

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

잘 봤습니다 ! 질문이 있는데 previousData를 사용하지 않는 이유가 있을까요?

@Clearsu
Copy link
Collaborator Author

Clearsu commented Jan 4, 2024

잘 봤습니다 ! 질문이 있는데 previousData를 사용하지 않는 이유가 있을까요?

@joonho0410 PreviousData를 사용하면 isLoading이 초기 페이지 로드 이후에는 항상 true여서 로딩 컴포넌트을 표시할 수 없어서 어쩔 수 없이 꺼놨습니다😅 이전 데이터를 계속 살려놓는게 좋을 것 같지만 이 옵션을 키면 리그 변경 시 일시적으로 이전 리그 정보가 렌더링돼서 보기 좋지 않더라고요.

@Clearsu Clearsu merged commit 4efa83f into main Jan 4, 2024
1 check passed
@Clearsu Clearsu deleted the others/토너먼트-브라켓-및-스와이퍼-로딩-컴포넌트-적용-#1203 branch January 4, 2024 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joonho0410 joonho0410 joonho0410 approved these changes

Assignees

@Clearsu Clearsu

Labels
refactor 유지보수와 잠재적 오류를 방지하기 위한 refactor issue user 일반 사용자가 사용하는 서비스에서 발생한 issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Others] 토너먼트 브라켓 및 스와이퍼 로딩 컴포넌트 적용
2 participants
@Clearsu @joonho0410