Merge branch 'fix/notif_new_fetch_bug' of https://github.com/ernestwisniewski/lemmy...

diff --cc src/shared/utils.ts
index 9257a4e6c657a5986dfe5f1d13ad8e2571af170e,f9c554f8ad8013742b98a372d2778cb36b68859f..695ba1e6f392d910e258f44cead4972889928bb2
--- 1/src/shared/utils.ts
--- 2/src/shared/utils.ts
+++ b/src/shared/utils.ts
@@@ -630,21 -686,18 +630,18 @@@ export function notifyPrivateMessage(pm
  function notify(info: NotifyInfo, router: any) {
    messageToastify(info, router);
  
-   // TODO absolute nightmare bug, but notifs are currently broken.
-   // Notification.new will try to do a browser fetch ???
- 
-   // if (Notification.permission !== "granted") Notification.requestPermission();
-   // else {
-   //   var notification = new Notification(info.name, {
-   //     icon: info.icon,
-   //     body: info.body,
-   //   });
- 
-   //   notification.onclick = (ev: Event): any => {
-   //     ev.preventDefault();
-   //     router.history.push(info.link);
-   //   };
-   // }
+   if (Notification.permission !== "granted") Notification.requestPermission();
+   else {
+     var notification = new Notification(info.name, {
 -      ...{body: info.body},
 -      ...(info.icon && {icon: info.icon})
++      ...{ body: info.body },
++      ...(info.icon.isSome() && { icon: info.icon.unwrap() }),
+     });
+ 
+     notification.onclick = (ev: Event): any => {
+       ev.preventDefault();
+       router.history.push(info.link);
+     };
+   }
  }
  
  export function setupTribute() {