Merge pull request #129 from MetaMask/batch-request-improvements

chore: satinise batch request response to remove null values
MetaMask · Jul 17, 2024 · 1c91e8b · 1c91e8b
2 parents a1ad744 + c40df00
commit 1c91e8b
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/metamask-android-sdk/src/main/java/io/metamask/androidsdk/CommunicationClient.kt b/metamask-android-sdk/src/main/java/io/metamask/androidsdk/CommunicationClient.kt
@@ -300,8 +300,9 @@ internal class CommunicationClient(context: Context, callback: EthereumEventCall
             }
             EthereumMethod.METAMASK_BATCH.value -> {
                 val result = data.optString("result")
-                val results: List<String> = Gson().fromJson(result, object : TypeToken<List<String>>() {}.type)
-                completeRequest(id, Result.Success.Items(results))
+                val results: List<String?> = Gson().fromJson(result, object : TypeToken<List<String?>>() {}.type)
+                val sanitisedResults = results.filterNotNull()
+                completeRequest(id, Result.Success.Items(sanitisedResults))
             }
             else -> {
                 val result = data.optString("result")