Merge pull request #97 from MennoVS/master

fix Notification.close on PushNotifications
flauc · Nov 1, 2016 · f65f336 · f65f336
2 parents 247350a + 26f7d7a
commit f65f336
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/src/push-notifications.service.ts b/src/push-notifications.service.ts
@@ -27,12 +27,12 @@ export class PushNotificationsService {
         return new Observable((obs: any) => {
 
             if (!('Notification' in window)) {
-                obs.error('Notifications are not available in this envirement');
+                obs.error('Notifications are not available in this environment');
                 obs.complete();
             }
 
             if (this.permission !== 'granted') {
-                obs.error(`The user didn't granted you permission to send push notifications`);
+                obs.error(`The user hasn't granted you permission to send push notifications`);
                 obs.complete();
             }
 
@@ -42,10 +42,6 @@ export class PushNotificationsService {
             n.onclick = (e: any) => obs.next({notification: n, event: e});
             n.onerror = (e: any) => obs.error({notification: n, event: e});
             n.onclose = () => obs.complete();
-            n.close = () => {
-                n.close.bind(n);
-                obs.complete();
-            };
         });
     }