woowacourse-teams / 2023-emmsale

18 stars 2 forks source link

Feature/#839 행사 카카오 공유 기능 #888

Closed tmdgh1592 closed 7 months ago

tmdgh1592 commented 8 months ago

️⃣ 연관된 이슈

close : #839

📝 작업 내용

EventDetailViewModel.kt

// before
val eventId = stateHandle[EVENT_ID_KEY] ?: DEFAULT_EVENT_ID

init {
    fetchEvent()
    fetchIsScrapped()
}

// after
var eventId = stateHandle[EVENT_ID_KEY] ?: DEFAULT_EVENT_ID

init이 제거되었습니다.
EventDetailActivity.kt

fun onCreate() {
    fetchEvent()
    ...
}

private fun fetchEvent() {
        val eventId = intent.data?.getQueryParameter(EVENT_ID_KEY)?.toLong()
        if (eventId != null) viewModel.eventId = eventId

        viewModel.fetchEvent()
        viewModel.fetchIsScrapped()
    }

스크린샷 (선택)

스크린샷 2023-12-19 오후 1 11 17 스크린샷 2023-12-19 오후 1 11 38

예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)

예상 소요 시간 : 3시간 실제 소요 시간 : 3시간

💬 리뷰어 요구사항 (선택)

행사 공유 문구는 제가 임의로 정의했습니다. 더 좋은 문구가 있으시다면 추천해주세요~

tmdgh1592 commented 8 months ago

@ki960213 제보해주신 공유된 메시지 전체 부분을 클릭하면 팅기는 버그 수정 완료하였습니다 : ) 이 외에 리뷰 반영과 의견을 남겨놓았으니 확인부탁드립니다.