fix: update onError definition to match implementation (#3349)

Co-authored-by: olivier <[email protected]>
TheWidlarzGroup · Nov 16, 2023 · fdbd6a6 · fdbd6a6
1 parent abd73dc
commit fdbd6a6
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 11 deletions.
diff --git a/src/VideoNativeComponent.ts b/src/VideoNativeComponent.ts
@@ -241,11 +241,21 @@ export type OnReceiveAdEventData = Readonly<{
 }>;
 
 export type OnVideoErrorData = Readonly<{
-  errorString: string;
-  errorException: string;
-  errorStackTrace: string;
-  errorCode: string;
-  error: string;
+  error: OnVideoErrorDataDetails;
+  target?: number; // ios
+}>;
+
+export type OnVideoErrorDataDetails = Readonly<{
+  errorString?: string; // android
+  errorException?: string; // android
+  errorStackTrace?: string; // android
+  errorCode?: string; // android
+  error?: string; // ios
+  code?: number; // ios
+  localizedDescription?: string; // ios
+  localizedFailureReason?: string; // ios
+  localizedRecoverySuggestion?: string; // ios
+  domain?: string; // ios
 }>;
 
 export type OnAudioFocusChangedData = Readonly<{

diff --git a/src/types/events.ts b/src/types/events.ts
@@ -118,13 +118,22 @@ export type OnReceiveAdEventData = Readonly<{
 }>;
 
 export type OnVideoErrorData = Readonly<{
-  errorString: string;
-  errorException: string;
-  errorStackTrace: string;
-  errorCode: string;
-  error: string;
+  error: OnVideoErrorDataDetails;
+  target?: number; // ios
+}>;
+
+export type OnVideoErrorDataDetails = Readonly<{
+  errorString?: string; // android
+  errorException?: string; // android
+  errorStackTrace?: string; // android
+  errorCode?: string; // android
+  error?: string; // ios
+  code?: number; // ios
+  localizedDescription?: string; // ios
+  localizedFailureReason?: string; // ios
+  localizedRecoverySuggestion?: string; // ios
+  domain?: string; // ios
 }>;
-
 export type OnAudioFocusChangedData = Readonly<{
   hasAudioFocus: boolean;
 }>;