Merge pull request #3805 from tgxworld/avoid_constructing_desktop_notifications_on_mobile
FIX: Don't subscribe to Desktop Notifications on mobile.
This commit is contained in:
commit
864607e01f
|
@ -29,10 +29,6 @@ export default {
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
bus.subscribe("/notification-alert/" + user.get('id'), function(data){
|
|
||||||
onNotification(data, user);
|
|
||||||
});
|
|
||||||
|
|
||||||
bus.subscribe("/notification/" + user.get('id'), function(data) {
|
bus.subscribe("/notification/" + user.get('id'), function(data) {
|
||||||
const oldUnread = user.get('unread_notifications');
|
const oldUnread = user.get('unread_notifications');
|
||||||
const oldPM = user.get('unread_private_messages');
|
const oldPM = user.get('unread_private_messages');
|
||||||
|
@ -85,7 +81,13 @@ export default {
|
||||||
});
|
});
|
||||||
|
|
||||||
if (!Ember.testing) {
|
if (!Ember.testing) {
|
||||||
initDesktopNotifications(bus);
|
if (!Discourse.Mobile.mobileView) {
|
||||||
|
bus.subscribe("/notification-alert/" + user.get('id'), function(data){
|
||||||
|
onNotification(data, user);
|
||||||
|
});
|
||||||
|
|
||||||
|
initDesktopNotifications(bus);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue