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

#PAR-382 : 싱글, 대결모드 조회 API Combine 처리 + Shimmer Effect 적용 #227

Merged
merged 11 commits into from
Oct 12, 2023

Conversation

nohjunh
Copy link
Member

@nohjunh nohjunh commented Oct 11, 2023

Description

  1. 대결모드 단일 조회 API 호출 레이어 구축

  2. 싱글모드와 대결모드 조회 API 호출을 하나의 UseCase에서 처리하기 위해 combine 사용

  • 효율성을 떠나, combine으로 두 데이터를 묶어 처리하는 학습 관점에서 접근 시도
  • MyPageViewModel에서는 getRunningHistory()를 호출 -> GetRunningHistoryUseCase에서 싱글, 대결모드 기록 조회 API를 호출하고 Flow를 Combine한다. 이때 CombinedRunningHistory를 통해 두 결과 값을 하나의 data class로 묶는다.
  • RunningHistoryState에서 Loading, Success, LoadFailed 처리를 통해 UI 상태를 추적한다.
  • RunningHistoryState에서는 싱글과 대결 결과를 각각 처리하기에 List에 값이 있는지 없는지 모드에 따라 적절히 표시 가능

Implementation

2023-10-11.7.28.05.mov

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@nohjunh nohjunh added the ✨ 기능 기능을 구현한다. label Oct 11, 2023
@nohjunh nohjunh changed the title #PAR-382 : 싱글, 대결모드 조회 API Combine 연동 + Shimmer Effect 적용 #PAR-382 : 싱글, 대결모드 조회 API Combine 처리 + Shimmer Effect 적용 Oct 11, 2023
@nohjunh nohjunh merged commit 34a2dd1 into develop Oct 12, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ 기능 기능을 구현한다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant