@stream-io/video-react-native-sdk 1.20.1-alpha.2 → 1.20.1

package/dist/commonjs/version.js

@@ -4,5 +4,5 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.version = void 0;
-const version = exports.version = '1.20.1-alpha.1';
+const version = exports.version = '1.20.1';
 //# sourceMappingURL=version.js.map

package/dist/commonjs/version.js.map

@@ -1 +1 @@
-{"version":3,"names":["version","exports"],"sourceRoot":"../../src","sources":["version.ts"],"mappings":";;;;;;AAAO,MAAMA,OAAO,GAAAC,OAAA,CAAAD,OAAA,GAAG,gBAAgB","ignoreList":[]}
+{"version":3,"names":["version","exports"],"sourceRoot":"../../src","sources":["version.ts"],"mappings":";;;;;;AAAO,MAAMA,OAAO,GAAAC,OAAA,CAAAD,OAAA,GAAG,QAAQ","ignoreList":[]}

package/dist/module/version.js

@@ -1,2 +1,2 @@
-export const version = '1.20.1-alpha.1';
+export const version = '1.20.1';
 //# sourceMappingURL=version.js.map

package/dist/module/version.js.map

@@ -1 +1 @@
-{"version":3,"names":["version"],"sourceRoot":"../../src","sources":["version.ts"],"mappings":"AAAA,OAAO,MAAMA,OAAO,GAAG,gBAAgB","ignoreList":[]}
+{"version":3,"names":["version"],"sourceRoot":"../../src","sources":["version.ts"],"mappings":"AAAA,OAAO,MAAMA,OAAO,GAAG,QAAQ","ignoreList":[]}

package/dist/typescript/version.d.ts

@@ -1,2 +1,2 @@
-export declare const version = "1.20.1-alpha.1";
+export declare const version = "1.20.1";
 //# sourceMappingURL=version.d.ts.map

package/dist/typescript/version.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"version.d.ts","sourceRoot":"","sources":["../../src/version.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,OAAO,mBAAmB,CAAC"}
+{"version":3,"file":"version.d.ts","sourceRoot":"","sources":["../../src/version.ts"],"names":[],"mappings":"AAAA,eAAO,MAAM,OAAO,WAAW,CAAC"}

package/ios/RTCViewPip.swift

@@ -13,6 +13,17 @@ class RTCViewPip: UIView {
     private var pictureInPictureController = StreamPictureInPictureController()
     private var webRtcModule: WebRTCModule?
     
+    override init(frame: CGRect) {
+        super.init(frame: frame)
+        setupNotificationObserver()
+        self.pictureInPictureController?.sourceView = self
+    }
+    
+    required init?(coder aDecoder: NSCoder) {
+        super.init(coder: aDecoder)
+        setupNotificationObserver()
+    }
+    
     private func setupNotificationObserver() {
         NotificationCenter.default.addObserver(
             self,
@@ -22,6 +33,10 @@ class RTCViewPip: UIView {
         )
     }
     
+    deinit {
+        NotificationCenter.default.removeObserver(self)
+    }
+    
     func setWebRtcModule(_ module: WebRTCModule) {
         webRtcModule = module
     }
@@ -67,21 +82,16 @@ class RTCViewPip: UIView {
         self.pictureInPictureController = nil
     }
     
-    override func didMoveToSuperview() {
-        super.didMoveToSuperview()
-        if self.superview == nil {
-            print("PiP - RTCViewPip has been removed from its superview.")
-            NotificationCenter.default.removeObserver(self)
+    override func didMoveToWindow() {
+        super.didMoveToWindow()
+        let isVisible = self.superview != nil && self.window != nil;
+        if (!isVisible) {
+            // view is detached so we cleanup the pip controller
+            // taken from:  https://github.com/software-mansion/react-native-screens/blob/main/Example/ios/ScreensExample/RNSSampleLifecycleAwareView.m
             DispatchQueue.main.async {
                 NSLog("PiP - onCallClosed called due to view detaching")
                 self.onCallClosed()
             }
-        } else {
-            print("PiP - RTCViewPip has been added to a superview.")
-            setupNotificationObserver()
-            DispatchQueue.main.async {
-                self.pictureInPictureController?.sourceView = self
-            }
         }
     }
 }

package/ios/RTCViewPipManager.swift

@@ -9,21 +9,34 @@ import Foundation
 
 @objc(RTCViewPipManager)
 class RTCViewPipManager: RCTViewManager {
-    
+
+    // A cached RTCViewPip reference.
+    //
+    // Often, the view unmounts before the `onCallClosed` command arrives and as a consequence
+    // pipView.onCallClosed() method wasn't called, as the view can't be found in the ViewRegistry
+    // causing dangling PiP window with the last video frame frozen.
+    // Now, once this happens, instead forwarding the command to the view returned by the registry,
+    // we manually apply it to the cached view. The current setup allows only one PipView, so we
+    // don't have to introduce more complex view tracking mechanism.
+    private var _view: RTCViewPip? = nil
+
     override func view() -> UIView! {
         let view = RTCViewPip()
         view.setWebRtcModule(self.bridge.module(forName: "WebRTCModule") as! WebRTCModule)
+        self._view = view
         return view
     }
-    
+
     override static func requiresMainQueueSetup() -> Bool {
         return true
     }
-    
+
     @objc func onCallClosed(_ reactTag: NSNumber) {
         self.bridge!.uiManager.addUIBlock { (_: RCTUIManager?, viewRegistry: [NSNumber: UIView]?) in
             guard let pipView = viewRegistry?[reactTag] as? RTCViewPip else {
-                NSLog("PiP - onCallClosed cant be called, Invalid view returned from registry, expecting RTCViewPip")
+                NSLog("PiP - onCallClosed can't be called, Invalid view returned from registry, expecting RTCViewPip. Disposing the cached view.")
+                self._view?.onCallClosed()
+                self._view = nil
                 return
             }
             DispatchQueue.main.async {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stream-io/video-react-native-sdk",
-  "version": "1.20.1-alpha.2",
+  "version": "1.20.1",
   "description": "Stream Video SDK for React Native",
   "author": "https://getstream.io",
   "homepage": "https://getstream.io/video/docs/react-native/",
@@ -125,9 +125,9 @@
     "@react-native-firebase/app": "^22.1.0",
     "@react-native-firebase/messaging": "^22.1.0",
     "@react-native/babel-preset": "^0.79.2",
-    "@stream-io/noise-cancellation-react-native": "workspace:^",
+    "@stream-io/noise-cancellation-react-native": "^0.2.0",
     "@stream-io/react-native-webrtc": "^125.4.0",
-    "@stream-io/video-filters-react-native": "workspace:^",
+    "@stream-io/video-filters-react-native": "^0.6.0",
     "@testing-library/jest-native": "^5.4.3",
     "@testing-library/react-native": "13.2.0",
     "@tsconfig/node14": "14.1.3",
@@ -164,4 +164,4 @@
       "typescript"
     ]
   }
-}
+}

package/src/version.ts

@@ -1 +1 @@
-export const version = '1.20.1-alpha.1';
+export const version = '1.20.1';

package/ios/StreamVideoReactNative.xcodeproj/xcuserdata/santhoshvaiyapuri.xcuserdatad/xcschemes/xcschememanagement.plist

@@ -1,14 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
-<plist version="1.0">
-<dict>
-	<key>SchemeUserState</key>
-	<dict>
-		<key>StreamVideoReactNative.xcscheme_^#shared#^_</key>
-		<dict>
-			<key>orderHint</key>
-			<integer>0</integer>
-		</dict>
-	</dict>
-</dict>
-</plist>