phlux/mattermostest
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix desktop notification not working with Mattermost 2.0

Browse Source 
Fix #36
Notification.permission was missing in override
This commit is contained in:
Yuya Ochiai
2016-02-20 21:41:00 +09:00
parent 6748078a83
commit 81baf05c78
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/browser/webview/mattermost.js
View File

@@ -115,6 +115,7 @@ function overrideNotificationWithBalloon() {
options: options options: options
}); });
}; };
Notification.permission = NativeNotification.permission;
Notification.requestPermission = function(callback) { Notification.requestPermission = function(callback) {
callback('granted'); callback('granted');
}; };
@@ -126,6 +127,7 @@ function overrideNotification() {
Notification = function(title, options) { Notification = function(title, options) {
this.notification = new NativeNotification(title, options); this.notification = new NativeNotification(title, options);
}; };
Notification.permission = NativeNotification.permission;
Notification.requestPermission = function(callback) { Notification.requestPermission = function(callback) {
callback('granted'); callback('granted');
}; };