Merge pull request #107 from laitlhadj/master

Add include_external_user_ids notification parameter
norkunas · Dec 21, 2018 · 333dc91 · 333dc91
1 parent b552b7e
commit 333dc91
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/Resolver/NotificationResolver.php b/src/Resolver/NotificationResolver.php
@@ -60,6 +60,8 @@ public function resolve(array $data)
             ->setAllowedTypes('include_ios_tokens', 'array')
             ->setDefined('include_android_reg_ids')
             ->setAllowedTypes('include_android_reg_ids', 'array')
+            ->setDefined('include_external_user_ids')
+            ->setAllowedTypes('include_external_user_ids', 'array')
             ->setDefined('include_wp_uris')
             ->setAllowedTypes('include_wp_uris', 'array')
             ->setDefined('include_wp_wns_uris')

diff --git a/tests/OneSignal/Tests/Resolver/NotificationResolverTest.php b/tests/OneSignal/Tests/Resolver/NotificationResolverTest.php
@@ -44,6 +44,7 @@ public function testResolveWithValidValues()
             'include_player_ids' => ['value'],
             'include_ios_tokens' => ['value'],
             'include_android_reg_ids' => ['value'],
+            'include_external_user_ids' => ['value'],
             'include_wp_uris' => ['value'],
             'include_wp_wns_uris' => ['value'],
             'include_amazon_reg_ids' => ['value'],
@@ -139,6 +140,7 @@ public function wrongValueTypesProvider()
             [['include_player_ids' => 'wrongType']],
             [['include_ios_tokens' => 'wrongType']],
             [['include_android_reg_ids' => 666]],
+            [['include_external_user_ids' => 666]],
             [['include_wp_uris' => 666]],
             [['include_wp_wns_uris' => 666]],
             [['include_amazon_reg_ids' => 666]],