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

MVVM - Repository, ViewModel구현 #3

Open
wants to merge 26 commits into
base: develop
Choose a base branch
from
Open

Conversation

Leesin0222
Copy link
Member

@Leesin0222 Leesin0222 commented Jan 3, 2025

PR 작업 요약

2주차 과제 중 MVVM패턴에 대해 구현해주었습니다. repository를 만들어주었고, viewModel을 구현해주었습니다.

작업 내용

  • ViewModel 구현
    • 화면 별로 ViewModel과 ViewModelFactory를 구현해주었습니다.
  • 모든 Activity, Fragment에서 ViewModel을 사용하도록 수정해주었습니다.
  • 카드 상세보기 화면의 카드정보를 불러오는 방식 변경
    • ASIS : 카드 상세 보기 화면에서 기존에는 이전 화면인 전체 카드 리스트화면에서 불러온 Card값을 카드 상세 보기 Activity에 intent로 넘겨주어 카드 정보를 뷰에 보여주는 식으로 구현해주었었습니다.
    • TOBE : 카드 상세 보기 화면에서 이전 화면인 전체 카드 리스트화면에서 불러온 Card id값만을 넘겨주어 changeCardReadStatus api호출 시 return해주는 Card정보를 사용하도록 수정해주었습니다.

Preview

Screen_recording_20250103_235731.webm

추가 사항

  • M(Model) V(Activity, Fragment) VM(ViewModel)
  • imageUrl이 빈 String이기 때문에 이미지가 preview영상 내에서 비어있습니다.

cardApi구현 및 httpLoggingInterceptor추가, converterFactory추가
@Leesin0222 Leesin0222 added the enhancement New feature or request label Jan 3, 2025
@Leesin0222 Leesin0222 self-assigned this Jan 3, 2025
Copy link

sonarqubecloud bot commented Jan 3, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarQube Cloud

} else {
break
}
}

Choose a reason for hiding this comment

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

Q) 이 부분의 문제를 파악하고, 개선할 수 있는 부분에 대해서 정리해주세요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants