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

DEP-214 feat: 짝꿍 만들기 / 기본 화면 -> 1단계 -> 2단계 기능 구현 #86

Merged
merged 8 commits into from
Dec 20, 2022

Conversation

juhwankim-dev
Copy link
Member

💁‍♂️ 변경 내용

KakaoTalk_20221220_034147058
KakaoTalk_20221220_034147235
KakaoTalk_20221220_034147390

AS-IS

  • 짝꿍이 없을 때 화면에 일러스트가 없었습니다.
  • 짝꿍과 함께하기 버튼을 눌러도 화면이 이동 되지 않았습니다.
  • 1단계 화면에서 습관 리스트를 불러오지 않았습니다.
  • 아이템을 클릭해도 클릭 효과가 없었습니다.
  • 다음 버튼을 눌러도 이동 되지 않았습니다.

TO-BE

  • 짝꿍이 없을 때 화면에 일러스트가 생겼습니다.
  • 짝꿍과 함께하기 버튼을 누르면 1단계 화면으로 이동 됩니다.
  • 1단계 화면에서 습관 리스트를 불러옵니다. (api 호출)
  • 아이템을 클릭하면 클릭 효과가 있습니다. (배경, 보더)
    • 단, mock 서버에서 모든 아이템의 id를 0으로 보내주고 있어서 정상 작동하지 않습니다.
  • 다음 버튼을 누르면 2단계 페이지로 이동합니다.

📢 전달사항

  • 습관 리스트를 불러오는 api에 status가 생겨서 기존 코드를 손봤습니다.
    status를 하드코딩으로 넣어주는 usecase가 2개 생겼습니다.

@juhwankim-dev juhwankim-dev added the enhancement New feature or request label Dec 19, 2022
@juhwankim-dev juhwankim-dev self-assigned this Dec 19, 2022
@juhwankim-dev juhwankim-dev requested a review from a team as a code owner December 19, 2022 18:50
private val habitRepository: HabitRepository
) {
suspend operator fun invoke(): Flow<DataState<List<Habit>>> {
return habitRepository.getHabits()
return habitRepository.getHabits(status = "ACTIVE")
Copy link
Member

Choose a reason for hiding this comment

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

Domain 모듈에서는 status enum 으로 적용하면 좋을거같습니다.
ACTIVE, ARCHIVED 외에 UNKNWON 도 가지게하면, 이후에 서버가 새로 값 추가해도 파싱 실패하지 않으면서 처리할수 있을거같은데요.
서버랑 직접 연동하는 dto 는 string 이어도 좋고, mapper 정도에서 변환해주면 좋을거같습니다.

Copy link
Member Author

Choose a reason for hiding this comment

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

(말씀하신 거대로 정확히 한 건지 헷갈리긴 한데..) 적용했습니다!

import com.depromeet.threedays.mypage.MyPageFragment
import dagger.hilt.android.AndroidEntryPoint

@AndroidEntryPoint
class MainActivity : BaseActivity<ActivityMainBinding>(R.layout.activity_main) {
val hasMate = false // TODO: 나중에 api 호출로 유무 판단
Copy link
Member

Choose a reason for hiding this comment

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

mate 조회하는 usecase 까지는 만들 수 있을거같습니다.

  • 현재는 habits?status=ACTIVE 중에서 it.mate != null 인 습관이 하나라도 있는지, 없는지로 구분하면 될거같고
  • api 추가되면 usecase 구현만 바꿔주면 될거같아요

Copy link
Member Author

Choose a reason for hiding this comment

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

감사합니다!! 이후에 pr때 api가 안나오면 적용해두겠습니다

@sonarcloud
Copy link

sonarcloud bot commented Dec 20, 2022

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 9 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@juhwankim-dev juhwankim-dev merged commit 137d748 into develop Dec 20, 2022
@juhwankim-dev juhwankim-dev deleted the feature/DEP-214_create_mate_func branch December 20, 2022 15:38
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