Merge pull request #146 from norkunas/add-app-url-option

Add app_url option in NotificationResolver

Author: norkunas

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## Version 2.3.0
+
+- Add app_url option in NotificationResolver
+
 ## Version 2.2.0
 
 - Add channel_for_external_user_ids option in NotificationResolver

src/Resolver/NotificationResolver.php

@@ -164,6 +164,8 @@ public function resolve(array $data): array
             ->setAllowedValues('web_url', function (string $value) {
                 return $this->filterUrl($value);
             })
+            ->setDefined('app_url')
+            ->setAllowedTypes('app_url', 'string')
             ->setDefined('send_after')
             ->setAllowedTypes('send_after', DateTimeInterface::class)
             ->setNormalizer('send_after', function (Options $options, DateTimeInterface $value) {

tests/Resolver/NotificationResolverTest.php

@@ -92,6 +92,7 @@ public function testResolveWithValidValues(): void
             'firefox_icon' => 'value',
             'url' => 'http://url.com',
             'web_url' => 'http://url.com',
+            'app_url' => 'myapp://path',
             'send_after' => new DateTime(),
             'delayed_option' => 'timezone',
             'delivery_time_of_day' => new DateTime(),
@@ -188,6 +189,7 @@ public function wrongValueTypesProvider(): iterable
         yield [['firefox_icon' => 666]];
         yield [['url' => 666]];
         yield [['web_url' => 666]];
+        yield [['app_url' => 666]];
         yield [['send_after' => 666]];
         yield [['delayed_option' => 666]];
         yield [['delivery_time_of_day' => 666]];