[Feat] #259 - 개인 코스 공유 기능을 추가 하였습니다. #261

thingineeer · 2024-03-11T07:54:55Z

🌱 작업한 내용

Scene 분기 처리
공유 함수 extension으로 분기 (for Reusable)
개인 코스 공유(firebase 기반)기능 추가

🌱 PR Point

🔥 이제 개인 코스도 타인에게 코스 공유를 할 수 있습니다.

🍀 privateCourse에 공유 기능을 추가 하여 보관함에 있는 코스도 공유할 수 있게 하였습니다.

SceneDelegate로 분기 추가
공유 코스인지, 아닌지 판별하기 위해 CourseType을 나눠주었습니다.

enum CourseType {
    case publicCourse, privateCourse
}

switch courseType {
case .publicCourse:
    let courseDetailVC = CourseDetailVC()
    courseDetailVC.getUploadedCourseDetail(courseId: courseId) // 공유 코스
    navigationController.pushViewController(courseDetailVC, animated: false)
case .privateCourse:
    let privateCourseDetailVC = RunningWaitingVC()
    privateCourseDetailVC.setData(courseId: courseId, publicCourseId: nil) // 개인 코스
    navigationController.pushViewController(privateCourseDetailVC, animated: false)
}

🍀 공유 기능 extension으로 분리 (shareCourse 함수)

문서화하였기 때문에, 추후에 읽고 가져다 사용만 하시면 됩니다.

아래 RunningWaitingVC 구현 부 에서 확인해서 사용하시면 됩니다.

🍀 RunningWaitingVC() 에 공유 버튼 로직 추가

@objc private func shareButtonDidTap() {
    guard let model = self.courseModel else {
        return
    }
    analyze(buttonName: GAEvent.Button.clickShare)
    
    self.shareCourse(
        courseTitle: model.title,
        courseId: model.id,
        courseImageURL: model.image,
        minimumAppVersion: "2.0.1",
        descriptionText: "이 코스는 링크로만 들어올 수 있어요!",
        parameter: "privateCourseId"
    )
}

📸 스크린샷

공유 코스가 내 코스 인 경우	공유 코스가 타인 코스 인 경우

카카오톡 공유 링크	카카오톡 공유 링크 상세

📮 관련 이슈

Resolved: [Feat] #259 - 개인 코스 공유 기능 추가 #259

- 같은 코드들이 반복되어 재사용 가능한 함수를 구현하였습니다. 확장성을 고려하여 앞으로 코스 관련 공유 기능이 생긴다면 아래 "shareCourse" 함수만 사용하면 됩니다.

513sojin

공유 기능 익스텐션으로 빼서 재사용성 무한대로 높여버리는 이명진 만만세요 그냥 ~ 너무 수고하셨습니다 !!!!

513sojin · 2024-03-11T13:30:16Z

Runnect-iOS/Runnect-iOS/Global/Extension/UIKit+/UIViewController+.swift

+    func shareCourse(
+        courseTitle: String,
+        courseId: Int,
+        courseImageURL: String,
+        minimumAppVersion: String,
+        descriptionText: String? = nil,
+        parameter: String
+    ) {


익스텐션으로 따로 빼두는게 좋을 것 같다는 생각을 했는데 역시 명진쌤,,,,, 👍🏻

513sojin · 2024-03-11T13:40:22Z

Runnect-iOS/Runnect-iOS/Global/Supports/SceneDelegate.swift

@@ -106,17 +113,26 @@ class SceneDelegate: UIResponder, UIWindowSceneDelegate {
        // to restore the scene back to its current state.
    }

-    func handleDynamicLink(_ dynamicLink: DynamicLink?) -> String? {
+    func handleDynamicLink(_ dynamicLink: DynamicLink?) -> (courseType: CourseType, courseId: Int)? {


guard let link = URL(string: "\(dynamicLinksDomainURIPrefix)/?\(courseParameter)=\(courseId)")
이 부분에서 courseParameter를 지정해준거에 따라 추출해서 courseType을 결정해주는 것으로 이해했는데 맞을까용 ?

맞습니다. 아래와 같은 코드로 URL Scheme에 해당하는 요소를 빼고 어떤 건지 구별하여 타입을 구분하여 리턴을 하도록 만들었습니다.

item.name == "courseId", item.name == "privateCourseId"

Runnect-iOS/Runnect-iOS/Global/Supports/SceneDelegate.swift

513sojin · 2024-03-11T13:47:11Z

Runnect-iOS/Runnect-iOS/Presentation/Running/VC/RunningWaitingVC.swift

+    private func isMyCourse() {
+        guard let isMyCourse = courseModel?.isNowUser else { return }
+
+        // 자기 코스가 아니라면 <공유, 더보기 버튼> 히든 처리
+        if !isMyCourse {
+            self.shareButton.isHidden = true
+            self.moreButton.isHidden = true
+        }


본인 코스인지 아닌지 처리하는 부분 확인했습니다~

513sojin · 2024-03-11T13:47:43Z

Runnect-iOS/Runnect-iOS/Presentation/Running/VC/RunningWaitingVC.swift

+        self.courseTitle = courseModel.title
+        self.naviBar.setTitle(self.courseTitle ?? "Test Code")


setData 함수 내에서 처리하던 부분을 여기에서 설정하게 된 이유가 있나요 ?! 단순 궁금증입니닷

이렇게 하면 Scene에서도 타이틀 관리를 할 수 있기 때문입니다~!
setData()를 통해 데이터를 넣어주면 title까지 모델에서 받아 올 수 있어요!

바꾸기 전에서는 Scene에서 타이틀을 어떻게 넣어줘야 할지 고민을 했었는데, 이렇게 바꿔줌으로써 해결을 할 수 있었습니다.

513sojin · 2024-03-11T13:56:17Z

Runnect-iOS/Runnect-iOS/Global/Supports/SceneDelegate.swift

+enum CourseType {
+    case publicCourse, privateCourse
+}


sceneDelegate의 역할이 scene의 라이프 사이클을 관리하는 거라서 이 파일 내부에서 CourseType에 대한 정의를 하는게 저희를 제외한 외부한테는 조금 이해하기 어려울 수 있겠다는 생각이 들엇습니다... 장황하게 말하긴 했지만 위치를 옮기거나(근데 옮길 곳이 없는듯..?) 주석을 다는게 어떨까용?!

주석을 추가 하도록 하겠습니다!

thingineeer

필요 없는 주석 제거
필요한 주석 추가

thingineeer · 2024-03-11T14:36:29Z

Runnect-iOS/Runnect-iOS/Presentation/Running/VC/RunningWaitingVC.swift

+        self.courseTitle = courseModel.title
+        self.naviBar.setTitle(self.courseTitle ?? "Test Code")


이렇게 하면 Scene에서도 타이틀 관리를 할 수 있기 때문입니다~!
setData()를 통해 데이터를 넣어주면 title까지 모델에서 받아 올 수 있어요!

바꾸기 전에서는 Scene에서 타이틀을 어떻게 넣어줘야 할지 고민을 했었는데, 이렇게 바꿔줌으로써 해결을 할 수 있었습니다.

thingineeer · 2024-03-11T14:38:18Z

Runnect-iOS/Runnect-iOS/Global/Supports/SceneDelegate.swift

@@ -106,17 +113,26 @@ class SceneDelegate: UIResponder, UIWindowSceneDelegate {
        // to restore the scene back to its current state.
    }

-    func handleDynamicLink(_ dynamicLink: DynamicLink?) -> String? {
+    func handleDynamicLink(_ dynamicLink: DynamicLink?) -> (courseType: CourseType, courseId: Int)? {


맞습니다. 아래와 같은 코드로 URL Scheme에 해당하는 요소를 빼고 어떤 건지 구별하여 타입을 구분하여 리턴을 하도록 만들었습니다.

item.name == "courseId", item.name == "privateCourseId"

thingineeer · 2024-03-11T14:39:46Z

Runnect-iOS/Runnect-iOS/Global/Supports/SceneDelegate.swift

+enum CourseType {
+    case publicCourse, privateCourse
+}


주석을 추가 하도록 하겠습니다!

- 필요 없는 주석 제거 - 필요한 주석 추가

thingineeer added 4 commits March 8, 2024 19:14

[Add] Runnect#259 - 개인 코스에 공유 버튼 추가

6da8786

[Feat] Runnect#259 - 개인 공유 기능 구현

02d54c1

[Feat] Runnect#259 - 링크로 들어오는 로직 분기 처리

f8c59f6

[Feat] Runnect#259 - 재사용 가능한 공유 함수 구현

32956c9

- 같은 코드들이 반복되어 재사용 가능한 함수를 구현하였습니다. 확장성을 고려하여 앞으로 코스 관련 공유 기능이 생긴다면 아래 "shareCourse" 함수만 사용하면 됩니다.

thingineeer added Feat 새로운 기능 구현 Refactor 전면 수정 명진😼 labels Mar 11, 2024

thingineeer requested a review from 513sojin March 11, 2024 07:54

thingineeer self-assigned this Mar 11, 2024

[Fix] Runnect#259 - 수출 관리 경고 제거

66fe2b6

513sojin approved these changes Mar 11, 2024

View reviewed changes

thingineeer commented Mar 11, 2024

View reviewed changes

thingineeer added 2 commits March 11, 2024 23:41

[Fix] Runnect#259 - 코드 리뷰 반영

c86e7fa

- 필요 없는 주석 제거 - 필요한 주석 추가

[Fix] Runnect#259 - isMyCourse() 가 바인딩이 안되는 이슈 해결

9d3a764

thingineeer force-pushed the #259---개인코스공유기능 branch from 54fb20c to 9d3a764 Compare March 11, 2024 15:46

thingineeer merged commit 80707e5 into Runnect:develop Mar 11, 2024
1 check passed

thingineeer deleted the #259---개인코스공유기능 branch March 11, 2024 15:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat] #259 - 개인 코스 공유 기능을 추가 하였습니다. #261

[Feat] #259 - 개인 코스 공유 기능을 추가 하였습니다. #261

thingineeer commented Mar 11, 2024 •

edited

Loading

513sojin left a comment

513sojin Mar 11, 2024

513sojin Mar 11, 2024

thingineeer Mar 11, 2024

513sojin Mar 11, 2024

513sojin Mar 11, 2024

thingineeer Mar 11, 2024

513sojin Mar 11, 2024

thingineeer Mar 11, 2024

thingineeer left a comment

thingineeer Mar 11, 2024

thingineeer Mar 11, 2024

thingineeer Mar 11, 2024

		self.courseTitle = courseModel.title
		self.naviBar.setTitle(self.courseTitle ?? "Test Code")

[Feat] #259 - 개인 코스 공유 기능을 추가 하였습니다. #261

[Feat] #259 - 개인 코스 공유 기능을 추가 하였습니다. #261

Conversation

thingineeer commented Mar 11, 2024 • edited Loading

🌱 작업한 내용

🌱 PR Point

🔥 이제 개인 코스도 타인에게 코스 공유를 할 수 있습니다.

🍀 privateCourse에 공유 기능을 추가 하여 보관함에 있는 코스도 공유할 수 있게 하였습니다.

🍀 공유 기능 extension으로 분리 (shareCourse 함수)

🍀 RunningWaitingVC() 에 공유 버튼 로직 추가

📸 스크린샷

📮 관련 이슈

513sojin left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

thingineeer left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

thingineeer commented Mar 11, 2024 •

edited

Loading