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

[FEATURE] Modified Match Detail Screen (1) #78

Merged
merged 16 commits into from
May 6, 2024
Merged

Conversation

eshc123
Copy link
Owner

@eshc123 eshc123 commented May 6, 2024

Description

  • 수정된 디자인에 따른 MatchDetailScreen UI 상단 부분 수정

Content

  1. MatchDetail에서 사용되는 type 필드를 enum class로 만들었습니다. 해당 필드는 서버, 서버내 DB에서도 enum class 로 사용 중이고
    ENUM('GOAL', 'OWNGOAL', 'SUBSTITUTION', 'YELLOWCARD', 'REDCARD', 'YELLOWREDCARD', 'PENALTY', 'MISSED_PENALTY') 와 같습니다.
    이름을 동일하게 가져가는 것이 더 편하게 처리할 수 있을 것이라 생각되어 동일하게 가져갔습니다.
  • 관련하여 MatchMapper 도 변경되었습니다.
  1. 득점 기록 목록을 더 간편하게 가져올 수 있도록 List<MatchDetail>.getScoreHistoryList()를 추가하였습니다.
  2. 득점 기록 문자열을 더 간편하게 가져올 수 있도록 MatchDetailscoringRecordText 를 추가하였습니다.
  • 2,3 관련하여 MatchCard도 수정하였습니다.
  1. MatchCard 내 MatchLeagueInfo 가 Match Detail에서도 쓰이므로 DesignSystem 모듈에 빼두어 사용하였습니다.
  • 관련하여 MatchCard도 수정하였습니다.
  1. MatchDetailScreen 상단 부분을 새로운 디자인 리뉴얼에 맞게 변경하였습니다.
  2. common 모듈에 UiState를 추가하였습니다.

Result screenshot

Comment

  1. MatchDetailUiState에서 match는 GetMatchDetailUseCase 호출 이전에 이미 Navigation과정에서 전달받은 인자로 초기화되기 때문에 따로 UiState로 두지 않았습니다.

@eshc123 eshc123 requested a review from KanuKim97 May 6, 2024 02:54
@eshc123 eshc123 self-assigned this May 6, 2024
@@ -0,0 +1,21 @@
package com.eshc.goonersapp.core.domain.model.match

enum class MatchDetailType {
Copy link
Collaborator

Choose a reason for hiding this comment

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

@eshc123 ! 질문이 있어요!
만약 OWNGOAL이 발생하게 되는 경우 기록은 상대팀에 표시되여야하는데 이 문제는 어떻게 처리해야할까요?

Copy link
Owner Author

Choose a reason for hiding this comment

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

홈팀 기준 어웨이팀 선수의 자책골이 있을 때 홈팀 Id 기준으로 필터링을 하면 자책골이 포함되어 있습니다.
teamId는 점수가 올라간 팀의 Id이고 선수 이름은 자책골을 넣은 선수로 나와서 PK와 마찬가지로 처리해주셔도 무방할 것 같습니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

넵! 확인했습니다! 감사합니다!

@KanuKim97 KanuKim97 merged commit ff61b4a into dev May 6, 2024
1 check passed
@eshc123 eshc123 deleted the feature/mod-match-screen branch May 27, 2024 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants