Skip to content

Commit 7f19633

Browse files
committed
more pr feedback
1 parent f1a5cc3 commit 7f19633

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

android/src/main/java/com/onesignal/rnonesignalandroid/RNUtils.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -316,7 +316,6 @@ private static Object convertValue(ReadableType type, ReadableMap map, String ke
316316
case String: return map.getString(key);
317317
case Map: return convertReadableMapToMap(map.getMap(key));
318318
case Array: return convertReadableArrayToList(map.getArray(key));
319-
default: throw new IllegalArgumentException("Unknown type for key: " + key);
320319
}
321320
}
322321

@@ -328,7 +327,6 @@ private static Object convertValue(ReadableType type, ReadableArray array, int i
328327
case String: return array.getString(index);
329328
case Map: return convertReadableMapToMap(array.getMap(index));
330329
case Array: return convertReadableArrayToList(array.getArray(index));
331-
default: throw new IllegalArgumentException("Unknown type for index: " + index);
332330
}
333331
}
334332

src/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1019,7 +1019,7 @@ export namespace OneSignal {
10191019
* Returns true if the value is a JSON-serializable object.
10201020
*/
10211021
function isObjectSerializable(value: unknown): boolean {
1022-
if (!(typeof value === 'object' && value !== null && !Array.isArray(value))) {
1022+
if (typeof value !== 'object' || value === null || Array.isArray(value)) {
10231023
return false;
10241024
}
10251025
try {

0 commit comments

Comments
 (0)