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

#85 [hotfix] 홈 뷰 로티 수정 #88

Merged
merged 11 commits into from
Jan 17, 2024

Conversation

stellar-halo
Copy link
Contributor

@stellar-halo stellar-halo commented Jan 17, 2024

📑 Work Description

  • 홈 뷰 로티 곰 타입에 맞게 안바뀌는 것 수정
  • 곰 타입 get,setUseCase 작성
  • 루틴 완료뷰에서 곰 타입에 맞게 이미지 변경

🛠️ Issue

📷 Screenshot

lottie.mp4

💬 To Reviewers

  • 최대한 서버 연결에서 반응을 줄이고자 곰 타입을 로컬에 저장하고 있습니다! UseCase부분 잘 봐주세요~

@stellar-halo stellar-halo added hotfix🚨 해결이 급한 부분 수정 강희🐬 강희가 작업함! labels Jan 17, 2024
@stellar-halo stellar-halo self-assigned this Jan 17, 2024
Copy link
Member

@minemi00 minemi00 left a comment

Choose a reason for hiding this comment

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

이른 시간까지 고생 많으셨습니다!!!

Comment on lines +29 to +32
override fun getBearType(): String = localDataSource.bearType
override fun setBearType(bearType: String) {
localDataSource.bearType = bearType
}
Copy link
Member

Choose a reason for hiding this comment

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

이런생각은 어떻게 하시나여..천재

Copy link
Contributor Author

Choose a reason for hiding this comment

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

코드는 어디선가 날라온 전부 남의 것,,,ㅎㅎ

private val authRepository: AuthRepository
) {
operator fun invoke(bearType: String) =
authRepository.setBearType(bearType)
Copy link
Member

Choose a reason for hiding this comment

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

유스케이스 set과 get 두개를 만드셨군요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

앞으로 곰돌이가 필요하다면 써주세요!

Copy link
Collaborator

@pump9918 pump9918 left a comment

Choose a reason for hiding this comment

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

꼼꼼한 체킹과 재사용성을 고려한 코드 멋집니다

}

private fun setDailyRoutineImage() {
val bearDrawableMap = mapOf(
Copy link
Collaborator

Choose a reason for hiding this comment

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

mapOf 손이 잘 안갔는데 확실히 깔끔하네요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저도 저번에 코리받고 수정했습니다!

@@ -85,11 +87,12 @@ class HappyProgressFragment :
doBtnColor = R.drawable.shape_main1_fill_12_rect,
backBtnAction = {},
doBtnAction = {
moveToHappyRoutineEmptyCardView()
startHappyRoutineCompleteActivity()
Copy link
Collaborator

Choose a reason for hiding this comment

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

로티 달성 뷰로 연결시켜주셨군요! 감사합니다

Copy link
Member

@emjayMJkim emjayMJkim left a comment

Choose a reason for hiding this comment

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

수고하셨습니당

Comment on lines +26 to +30
val bearDrawableMap = mapOf(
BROWN to R.drawable.ic_bear_handsup_brown,
GRAY to R.drawable.ic_bear_handsup_gray,
RED to R.drawable.ic_bear_handsup_red,
WHITE to R.drawable.ic_bear_handsup_panda
Copy link
Member

Choose a reason for hiding this comment

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

굿구웃 👍

@stellar-halo stellar-halo merged commit dd3fa2a into develop Jan 17, 2024
1 check passed
@stellar-halo stellar-halo deleted the feature/#85-hotfix-home-lottie branch January 17, 2024 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hotfix🚨 해결이 급한 부분 수정 강희🐬 강희가 작업함!
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[hotfix] 홈 뷰 곰 lottie 파일 수정
4 participants