feature #1853 [Notify] Added options to Notification (Sven Scholz)

This PR was merged into the 2.x branch.

Discussion
----------

[Notify] Added `options` to Notification

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| License       | MIT

Added body, icon, tag ,renotify for desktop notifications
This pull-request are need changes in src/Symfony/Component/Notifier/Bridge/Mercure/MercureOptions.php. I have made a pull request there too.

Commits
-------

056eb2c [Notify] Added body, icon, tag ,renotify to controller.js for mercure-notifier

Author: Kocal

src/Notify/CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 2.24.0
+
+-   Added `options` to Notification
+
 ## 2.13.2
 
 -   Revert "Change JavaScript package to `type: module`"

src/Notify/assets/dist/controller.d.ts

@@ -13,6 +13,6 @@ export default class extends Controller {
     initialize(): void;
     connect(): void;
     disconnect(): void;
-    _notify(content: string | undefined): void;
+    _notify(title: string | undefined, options: NotificationOptions | undefined): void;
     private dispatchEvent;
 }

src/Notify/assets/dist/controller.js

@@ -26,7 +26,10 @@ class default_1 extends Controller {
             return;
         }
         this.eventSources.forEach((eventSource) => {
-            const listener = (event) => this._notify(JSON.parse(event.data).summary);
+            const listener = (event) => {
+                const { summary, content } = JSON.parse(event.data);
+                this._notify(summary, content);
+            };
             eventSource.addEventListener('message', listener);
             this.listeners.set(eventSource, listener);
         });
@@ -42,17 +45,17 @@ class default_1 extends Controller {
         });
         this.eventSources = [];
     }
-    _notify(content) {
-        if (!content)
+    _notify(title, options) {
+        if (!title)
             return;
         if ('granted' === Notification.permission) {
-            new Notification(content);
+            new Notification(title, options);
             return;
         }
         if ('denied' !== Notification.permission) {
             Notification.requestPermission().then((permission) => {
                 if ('granted' === permission) {
-                    new Notification(content);
+                    new Notification(title, options);
                 }
             });
         }

src/Notify/assets/src/controller.ts

@@ -49,7 +49,12 @@ export default class extends Controller {
         }
 
         this.eventSources.forEach((eventSource) => {
-            const listener = (event: MessageEvent) => this._notify(JSON.parse(event.data).summary);
+            const listener = (event: MessageEvent) => {
+                const { summary, content } = JSON.parse(event.data);
+
+                this._notify(summary, content);
+            };
+
             eventSource.addEventListener('message', listener);
             this.listeners.set(eventSource, listener);
         });
@@ -70,19 +75,19 @@ export default class extends Controller {
         this.eventSources = [];
     }
 
-    _notify(content: string | undefined) {
-        if (!content) return;
+    _notify(title: string | undefined, options: NotificationOptions | undefined) {
+        if (!title) return;
 
         if ('granted' === Notification.permission) {
-            new Notification(content);
+            new Notification(title, options);
 
             return;
         }
 
         if ('denied' !== Notification.permission) {
             Notification.requestPermission().then((permission) => {
                 if ('granted' === permission) {
-                    new Notification(content);
+                    new Notification(title, options);
                 }
             });
         }