[Feat] #640 - 푸시알림별 화면 전환 #641

사용하지 않는 홈으로의 화면전환이나 @objc 키워드 붙여둔경우는 모든뷰에서 홈으로 갈때 사용하려고 하신 것 같은데 이때는 background 에서 푸시알림을 통해 앱을 런치하는 경우라고 생각이 들거든용?
이때 지금 스플래시에서 이동하는 것이 아니라서 스플래시에서 홈으로 화면 전환할 때 분기처리를 통해서 습관방으로의 화면전환을 추가해줘도 될거 같은생각이 들어요! 그래서 모든 뷰에서 노티를 받을 준비는 안하셔도 될 것 같아요!

hyun99999 · 2022-09-19T10:49:23Z

Spark-iOS/Spark-iOS/Source/AppDelegate.swift

+              let roomId: String = userInfo["roomId"] as? String else { return }
+        guard let mainTBC = UIStoryboard(name: Const.Storyboard.Name.mainTabBar, bundle: nil).instantiateViewController(withIdentifier: Const.ViewController.Identifier.mainTabBar) as? MainTBC else { return }
+
+        guard let window = UIApplication.shared.windows.first else { return }


windows 가 deprecated 된다고 하니 대체를 알아봐야할 것 같아요 (오-열)

제 windows 돌려주세요

에?? 그럼뭘주는거야

https://intrepidgeeks.com/tutorial/windows-was-deprecated-in-ios-150-ignore-warning
요거 참고해서 수정해두겠습니다 .. @hyun99999 @L-j-h-c

hyun99999 · 2022-09-19T10:49:55Z

Spark-iOS/Spark-iOS/Source/AppDelegate.swift

+        UIView.transition(with: window, duration: 0.5, options: .transitionCrossDissolve, animations: nil)
+
+        if recordId.isEmpty {
+            guard let nextVC = UIStoryboard(name: Const.Storyboard.Name.habitRoom, bundle: nil).instantiateViewController(withIdentifier: Const.ViewController.Identifier.habitRoom) as? HabitRoomVC else { return }


의미를 부여하기 위해서 nextVC 변수명을 habitRoomVC 로 변경해도 좋을 것 같아요!(물론 내가 짠 변수명임 ㅋㅋㅋㅋㅋㅋ쿠ㅜ 생각해보니 변경하면 좋을 것 같네여)

hyun99999 · 2022-09-19T10:58:52Z

Spark-iOS/Spark-iOS/Source/ViewControllers/Auth/AuthUpload/AuthUploadVC.swift

+    @objc
+    private func dismissToHomeVC() {
+//        switch vcType {
+//        case .photoTimer:
+//            self.dismiss(animated: true) {
+//                self.presentingViewController?.navigationController?.popViewController(animated: true)
+//            }
+//        case .photoOnly:
+//            self.dismiss(animated: true)
+//        default:
+//
+//        }
+        self.dismiss(animated: true) {
+            self.presentingViewController?.navigationController?.popViewController(animated: true)
+        }
+    }
+


분기처리가 유효해보이는뎅 테스트중인가용?

노티로 보냈을 때 뷰를 내리기 위해 사용했던 분기처리였는데,
AppDelegate나 SceneDelegate에서 루트를 변경해서 화면 전환하는 경우에는 필요 없어서 주석처리해뒀습니다!
삭제해버린다는 것을 깜빡했네요..

hyun99999 · 2022-09-19T11:00:32Z

Spark-iOS/Spark-iOS/Source/ViewControllers/TabBar/HomeVC.swift

+            if  UIApplication.getMostTopViewController() == self || notification.name == .startHabitRoom {
+                navigationController?.pushViewController(nextVC, animated: true)
+            } else {
+                self.popToHomeVC {
+                    self.navigationController?.pushViewController(nextVC, animated: true)
+                }
+            }
+        }
+    }


homeVC 에서 notification 을 받으면 homeVC 일텐데 else 로 빠지는 경우는 어떤 시나리오를 예상하고 고려해두셨을까용?

아 이것두 이전에 모두 노티 보내서 처리하려고 했을 때,
노티 보낸 당시 가장 맨 위의 뷰가 homeVC가 아니면(습관방이라던가 보관함, 피드 탭이라던가 보관함 상세라던가 등등)
홈으로 보낸 뒤 습관방으로 push 하기 위해 넣었던 부분인데
이제 필요없을 것 같습니당
삭제하겠습니다 !

hyun99999 · 2022-09-19T11:04:53Z

Spark-iOS/Spark-iOS/Source/ViewControllers/TabBar/MainTBC.swift

+    /// 푸시알림 종류에 따라 탭(feed/home) 이동
+    @objc
+    func showTab(_ notification: Notification) {
+        if let userInfo = notification.userInfo {
+            if let recordId = userInfo["recordID"] as? String {
+                selectedIndex = recordId.isEmpty ? 1 : 0
+            }
+        }
+    }


좋은 로직 같습니당! 🧑‍🏭

hyun99999 · 2022-09-19T11:54:45Z

// active, inactive, background enum 케이스로 존재해서 씬델리게이트 말구 앱델리게이트에서 처리해주어도 될거 같아요!
// 지금 씬 델리게이트 코드를 주석처리해도 피드뷰로 이동하네용 
// 만약에 분리할거라면 여기서는 active 랑 inactive 에서만 실행되도록 해줘야할거 같아요
let applicationState = UIApplication.shared.applicationState

if applicationState == .background {
     let info: [String: Any] = ["recordID": recordId, "roomID": roomId]

    DispatchQueue.main.asyncAfter(deadline: DispatchTime.now() + 2.0) {
        NotificationCenter.default.post(name: .pushNotificationTapped, object: nil, userInfo: info)
    }
}

// 씬델리게이트 코드를 유지할 것이라면 여기서 active, inactive 설정해주어야 제대로 분기처리가 될거 같아요!
if recordId.isEmpty {
     guard let nextVC = UIStoryboard(name: Const.Storyboard.Name.habitRoom, bundle: nil).instantiateViewController(withIdentifier: Const.ViewController.Identifier.habitRoom) as? HabitRoomVC else { return }
    nextVC.roomID = Int(roomId)
            
    DispatchQueue.main.asyncAfter(deadline: DispatchTime.now() + 0.5) {
        let topVC = UIApplication.getMostTopViewController()
         topVC?.navigationController?.pushViewController(nextVC, animated: true)
    }
} else {
     let info: [String: Any] = ["recordID": recordId, "roomID": roomId]
    NotificationCenter.default.post(name: .pushNotificationTapped, object: nil, userInfo: info)
}

- 서버에서 recordId 는 "" 빈문자열로 넘어오기 떄문에 nil 값을 가질 수 없다. 그래서 조건문자체가 먹히지 않았다. - 삭제한 조건문은 popToHomeVC 를 사용할 시나리오가 없을거라 생각하고 삭제했다.

yangsubinn · 2022-09-20T06:33:33Z

코드리뷰 반영 수정하고, #642 풀 받아서 푸시해뒀습니다 !
확인 부탁드립니다🙇‍♀️ @hyun99999

hyun99999 · 2022-09-21T00:56:58Z

Spark-iOS/Spark-iOS/Source/AppDelegate.swift

+            guard let windowScene = UIApplication.shared.connectedScenes.first as? UIWindowScene else { return }
+            let window = windowScene.windows.first
+            window?.rootViewController = mainTBC
+            UIView.transition(with: window ?? UIWindow(), duration: 0.5, options: .transitionCrossDissolve, animations: nil)


해결방법까지.. 코드 잘 보고 갑니다.
window 도 guard let 구문에서 옵셔널 바인딩 해주어도 좋을 것 같네용

이것까지 수정해서 머지할게여 !

hyun99999 · 2022-09-21T01:01:04Z

Spark-iOS/Spark-iOS/Source/ViewControllers/Auth/AuthTimer/AuthTimerVC.swift

+
+    @objc
+    private func dismissToHomeVC() {
+        self.dismiss(animated: true) {
+            self.presentingViewController?.navigationController?.popViewController(animated: true)
+        }
+    }


요 부분 아직 안지워진거 같아요!

아고 지워서 푸시하겠습니다!

yangsubinn added 2 commits September 18, 2022 12:23

[Feat] 푸시알림 탭했을 경우, notificationCenter로 화면 전환

c2fa2e1

[Fix] 습관방으로 화면 전환 수정 및 옵저버 삭제

bbcfcdd

yangsubinn added 🦹t없e맑은水빈 우리 선배 이젠 타이니하지 않아 🖐help_wanted🖐 도움이 필요해요~! labels Sep 18, 2022

yangsubinn requested a review from hyun99999 September 18, 2022 16:47

yangsubinn self-assigned this Sep 18, 2022

[Fix] foreground 상태에서 푸시알림 탭했을 경우, 모두 mainTBC를 rootVC로 변경

95eb71e

L-j-h-c approved these changes Sep 19, 2022

View reviewed changes

hyun99999 reviewed Sep 19, 2022

View reviewed changes

hyun99999 added 2 commits September 20, 2022 02:47

[Fix] TeamSparker#640 - application state 분기처리 수정

ffe1e83

hyun99999 mentioned this pull request Sep 19, 2022

[Feat] #640 - 푸시알림 카테고리별 화면 전환 #642

Merged

yangsubinn closed this Sep 20, 2022

yangsubinn reopened this Sep 20, 2022

yangsubinn added 3 commits September 20, 2022 15:14

[Chore] 코드리뷰 반영, 불필요한 코드 삭제

d3012c7

[Merge] TeamSparker#642 feature/TeamSparker#640 pull

44efec3

[Fix] windows deprecated 수정

245d6b5

hyun99999 approved these changes Sep 21, 2022

View reviewed changes

hyun99999 reviewed Sep 21, 2022

View reviewed changes

[Chore] window optional binding, 불필요한 함수 삭제

d6b309a

yangsubinn merged commit 8b1ec65 into TeamSparker:develop Sep 21, 2022

yangsubinn deleted the feature/#640 branch October 11, 2022 02:05

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat] #640 - 푸시알림별 화면 전환 #641

[Feat] #640 - 푸시알림별 화면 전환 #641

yangsubinn commented Sep 18, 2022

L-j-h-c left a comment

hyun99999 left a comment

hyun99999 Sep 19, 2022

yangsubinn Sep 20, 2022

L-j-h-c Sep 20, 2022

yangsubinn Sep 20, 2022

hyun99999 Sep 19, 2022

hyun99999 Sep 19, 2022

yangsubinn Sep 20, 2022

hyun99999 Sep 19, 2022

yangsubinn Sep 20, 2022

hyun99999 Sep 19, 2022

hyun99999 commented Sep 19, 2022 •

edited

Loading

yangsubinn commented Sep 20, 2022

hyun99999 Sep 21, 2022

yangsubinn Sep 21, 2022

hyun99999 Sep 21, 2022

yangsubinn Sep 21, 2022

[Feat] #640 - 푸시알림별 화면 전환 #641

[Feat] #640 - 푸시알림별 화면 전환 #641

Conversation

yangsubinn commented Sep 18, 2022

🔥Pull requests

🚨참고 사항

📸 스크린샷

📟 관련 이슈

L-j-h-c left a comment

Choose a reason for hiding this comment

hyun99999 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

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hyun99999 commented Sep 19, 2022 • edited Loading

yangsubinn commented Sep 20, 2022

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

hyun99999 commented Sep 19, 2022 •

edited

Loading