[FEAT/#364] 공고 상세 뷰 / 카카오톡 공유하기 by arinming · Pull Request #373 · teamterning/Terning-Android

arinming · 2025-04-01T15:28:32Z

closed [FEAT] 공고 상세 뷰 / 카카오톡 공유하기 기능 추가 #364

⛳️ Work Description

공유하기 버튼 추가
카카오 공유하기 커스텀 피드로 구현
카카오 공유하기 ViewModel 추가

📸 Screenshot

364.webm

📢 To Reviewers

카카오 공유하기 기능은 구현했고, 딥링크는 추후 구현할 예정입니다,..,✨

boiledeggg

너무 고생하셨어요👍 카카오 로직 관련해서 제 생각을 조금 달아봤는데 보시고 의견 남겨주세용~

boiledeggg · 2025-04-01T16:47:23Z

                        painter = painterResource(id = R.drawable.ic_back),
                        contentDescription = stringResource(id = R.string.ic_back),
-                        modifier = Modifier
+                        modifier = modifier


파라미터로 받는 modifier는 CenterAlignedTopAppBar의 수정자로 사용하는게 적합하다고 생각해요!
이렇게 내부 컴포넌트의 수정자로 사용하게 되면 외부에서 이 탑바를 호출하여 사용할 때 수정자로 인한 혼란이 생길 것 같네요,,

아공 맨날 요부분 실수하네여 ㅠ 섬세킹!!! 따봉석준!!!

boiledeggg · 2025-04-01T16:59:27Z

+private fun shareToKakaoTalk(context: Context, internInfo: InternInfo) {
+    val templateId = 118600L
+
+    val templateArgs = mapOf(
+        "COMPANY_IMG" to internInfo.companyImage,
+        "TITLE" to internInfo.title,
+        "DEADLINE" to internInfo.deadline,
+        "START_DATE" to internInfo.startYearMonth,
+        "PERIOD" to internInfo.workingPeriod,
+        "REGI_WEB_DOMAIN" to internInfo.url,
+        "A_E" to internInfo.url,
+        "I_E" to internInfo.url
+    )
+
+    if (ShareClient.instance.isKakaoTalkSharingAvailable(context)) {
+        ShareClient.instance.shareCustom(
+            context,
+            templateId,
+            templateArgs
+        ) { sharingResult, error ->
+            if (error != null) {
+                Timber.e("카카오톡 공유 실패: ${error.message}")
+            } else if (sharingResult != null) {
+                context.startActivity(sharingResult.intent)
+            }
+        }
+    } else {
+        val sharerUrl = WebSharerClient.instance.makeCustomUrl(templateId, templateArgs)
+        try {
+            KakaoCustomTabsClient.openWithDefault(context, sharerUrl)
+        } catch (e: Exception) {
+            Timber.e("웹 공유 실패: ${e.message}")
+        }
+    }
+}


개인적으로 카카오 관련 로직은 따로 designsystem/util에 클래스로 만들거나, 카카오 모듈 또는 써드파티 라이브러리를 위한 모듈을 만들어서 관리하면 좋을 것 같아요!

InternViewModel에서 templateArgs을 만들어서 카카오 공유하기 Util 클래스의 함수를 호출하는 식으로 풀면 될 것 같고, Context는 ApplicationContext를 힐트로 주입받는 식으로 해결할 수 있을 것 같네요!

이렇게 하면 이 로직을 뷰모델에서 관리할 수 있지 않을까 추측해봅니다😄

좋은 의견인 것 같아요!🚀
카카오 관련 클래스를 하나 만들면 제가 카카오 로그인에 사용했던 부분들도 그 쪽으로 옮길 수 있을 것 같네용

헉 ㅠ 동의합니다
이렇게 리팩토링하니 로직 분리가 되어서 훨씬 깔끔하고 유지보수하기 좋아졌네요
아직 이 코드에 딥링크 적용하기엔 너무 복잡한 것 같아서 임시방편으로 공유 되는 것까지 확인했습니다! 감사해용오 ㅜㅜ!!!!

boiledeggg · 2025-04-01T17:01:13Z

+    data class Toast(@StringRes val message: Int) :
+        InternSideEffect()
+
+    data class ShareKakaoTalk(val internInfo: InternInfo) : InternSideEffect()


위처럼 바꾸면 ShareKakaoTalk SideEffect도 불필요해지겠네요!

boiledeggg · 2025-04-01T17:01:57Z

+    fun shareWithKakaoTalk(
+        internInfo: InternInfo
+    ) {
+        viewModelScope.launch {
+            _sideEffect.emit(InternSideEffect.ShareKakaoTalk(internInfo))
+        }
+    }


여기에서 바로 카카오 공유하기를 실행시키는 그런 그림?

Hyobeen-Park

수고하셨습니다!! 최고에욥😊

Hyobeen-Park · 2025-04-02T15:33:54Z

                    navController.popBackStack()
                },
+                actions = listOf(
+                    {},


요기 빈 람다 없어도 되는거면 지우는게 좋을 것 같아요!

생각해보니 탑바 액션 리스트에서 이제 drop 1을 할 필요가 없는데 나이스캐치네요 ㅠㅠ 싹다 리팩토링ㅎ ㅐ주었습니다

Hyobeen-Park · 2025-04-02T15:47:07Z

+                                    .noRippleClickable {
+                                        onClickShareButton()
+                                    }


onClick 함수가 2번 적용되는 것 같아요! Icon 내부에서 클릭 이벤트를 지정해준다면 IconButton을 사용하지 않아도 괜찮지 않을까 싶네요😊

Suggested change

.noRippleClickable {

onClickShareButton()

}

.noRippleClickable(onClick = onClickShareButton)

그리고 람다를 새로 생성하면 객체가 하나 추가되는거라 미세하게나마 성능에 영향을 줄 수 있을 것 같아요! onClickShareButton만 실행하는 것이라면 함수 그대로 바로 넘겨주는 것은 어떨까요?

가독성이 훨씬 좋아졌읍니다 최고 ㅠㅠ!

leeeyubin

수고하셨어요!! 딥링크도 아자아자..🔥

leeeyubin · 2025-04-03T10:28:59Z

+private fun shareToKakaoTalk(context: Context, internInfo: InternInfo) {
+    val templateId = 118600L
+
+    val templateArgs = mapOf(
+        "COMPANY_IMG" to internInfo.companyImage,
+        "TITLE" to internInfo.title,
+        "DEADLINE" to internInfo.deadline,
+        "START_DATE" to internInfo.startYearMonth,
+        "PERIOD" to internInfo.workingPeriod,
+        "REGI_WEB_DOMAIN" to internInfo.url,
+        "A_E" to internInfo.url,
+        "I_E" to internInfo.url
+    )
+
+    if (ShareClient.instance.isKakaoTalkSharingAvailable(context)) {
+        ShareClient.instance.shareCustom(
+            context,
+            templateId,
+            templateArgs
+        ) { sharingResult, error ->
+            if (error != null) {
+                Timber.e("카카오톡 공유 실패: ${error.message}")
+            } else if (sharingResult != null) {
+                context.startActivity(sharingResult.intent)
+            }
+        }
+    } else {
+        val sharerUrl = WebSharerClient.instance.makeCustomUrl(templateId, templateArgs)
+        try {
+            KakaoCustomTabsClient.openWithDefault(context, sharerUrl)
+        } catch (e: Exception) {
+            Timber.e("웹 공유 실패: ${e.message}")
+        }
+    }
+}


좋은 의견인 것 같아요!🚀
카카오 관련 클래스를 하나 만들면 제가 카카오 로그인에 사용했던 부분들도 그 쪽으로 옮길 수 있을 것 같네용

# Conflicts: # feature/intern/src/main/java/com/terning/feature/intern/InternRoute.kt # feature/intern/src/main/java/com/terning/feature/intern/InternViewModel.kt # feature/intern/src/main/java/com/terning/feature/intern/InternViewSideEffect.kt

…364-share-kakao

boiledeggg

너무 고생많으셨습니다!! 마지막 딥링크까지 화이팅💪

boiledeggg · 2025-04-22T06:09:58Z

 dependencies {
    // domain
    implementation(projects.domain.intern)
+    implementation(libs.kakao.share)


카카오 util 클래스를 만들었으니 이 의존성은 없어도 되지 않나요??

나이스 캐치 감사합니당!

boiledeggg · 2025-04-22T06:12:28Z

boiledeggg · 2025-04-22T06:12:57Z

+class KakaoUtil @Inject constructor(
+    @ApplicationContext private val context: Context
+) {
+    private val templateId = 118600L


이 templateId는 무슨 기준으로 넣는건가요?? 단순 궁금증ㅎ

카카오 공유하기 메시지를 커스텀으로 만들때 식별하는 ID인데 일단 임의로 넣어둔 값이고 지금은 의미가 없어서 ㅜㅜ 다음 feature에서 유동적으로 수정하려고 합니당!

leeeyubin

확인했습니다! 만약에 저도 로그인 로직 구현을 위해 이 KakaoUtil 클래스를 사용한다면 따로 인터페이스를 만들어서 역할을 분리할 수 있게 만들어야겠네요!

arinming added 7 commits March 24, 2025 18:31

[ADD/#364] manifest 추가

7ad80c9

[ADD/#364] 공유하기 버튼 추가

3974c5a

[ADD/#364] 라이브러리 추가

24ac877

[FEAT/#364] 카카오 공유하기 구현

6eba5ae

[FEAT/#364] 카카오 공유하기 ViewModel 추가

a41ae24

[FEAT/#364] 카카오 공유하기 커스텀 피드로 변경

d756d2f

[FEAT/#364] 뒤로가기 회색 배경 삭제

67f8a29

arinming added UI 💐 UI 작업 아린💛 아린 labels Apr 1, 2025

arinming added this to the 4차 스프린트 작업 milestone Apr 1, 2025

arinming requested review from Hyobeen-Park, boiledeggg and leeeyubin April 1, 2025 15:28

arinming self-assigned this Apr 1, 2025

arinming linked an issue Apr 1, 2025 that may be closed by this pull request

[FEAT] 공고 상세 뷰 / 카카오톡 공유하기 기능 추가 #364

Closed

2 tasks

arinming added this to Terning-Android Apr 1, 2025

boiledeggg reviewed Apr 1, 2025

View reviewed changes

Hyobeen-Park reviewed Apr 2, 2025

View reviewed changes

leeeyubin approved these changes Apr 3, 2025

View reviewed changes

arinming added 9 commits April 6, 2025 19:14

Merge remote-tracking branch 'origin/develop' into feat/#364-share-kakao

ca45ac9

Merge remote-tracking branch 'origin/develop' into feat/#364-share-kakao

af08d07

Merge remote-tracking branch 'origin/feat/#364-share-kakao' into feat/#…

d9cb76e

…364-share-kakao

Merge remote-tracking branch 'origin/develop' into feat/#364-share-kakao

20bfc93

[FIX/#364] dev 머지 충돌 해결

fe74e77

[FIX/#364] UI 코드 리뷰 반영

3c4f0ff

[REFACTOR/#364] Kakao Util 추가하여 리팩토링

d4c6b23

[DELETE/#364] 코드 정리

f364a0d

arinming requested review from boiledeggg and leeeyubin April 21, 2025 13:38

arinming requested a review from Hyobeen-Park April 21, 2025 13:38

boiledeggg approved these changes Apr 22, 2025

View reviewed changes

leeeyubin approved these changes Apr 22, 2025

View reviewed changes

[DELETE/#364] 필요없는 gradle 삭제

b279d27

arinming merged commit eb82547 into develop Apr 23, 2025
1 check passed

github-project-automation Bot moved this to Done in Terning-Android Apr 23, 2025

Conversation

arinming commented Apr 1, 2025

⛳️ Work Description

📸 Screenshot

📢 To Reviewers

Uh oh!

boiledeggg left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

boiledeggg Apr 1, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Hyobeen-Park left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

leeeyubin left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

boiledeggg left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

leeeyubin left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

boiledeggg Apr 1, 2025 •

edited

Loading