[Feat] #259 - 개인 코스 공유 기능을 추가 하였습니다. by thingineeer · Pull Request #261 · Runnect/Runnect-iOS

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

    }

-    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

    }

-    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

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

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

Conversation

thingineeer commented Mar 11, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🌱 작업한 내용

🌱 PR Point

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

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

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

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

📸 스크린샷

📮 관련 이슈

Uh oh!

513sojin 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!

Uh oh!

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!

thingineeer 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

thingineeer commented Mar 11, 2024 •

edited

Loading