local-it/vikunja-frontend
0
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

fix: check if notifications are available at all before checking if triggered are available

Browse source 
Should fix sentry issue FRONTEND-OSS-J
This commit is contained in:
kolaente 2021-10-31 12:56:32 +01:00
parent 8bbdbe62a4
commit 8389587a60
No known key found for this signature in database
GPG key ID: F40E70337AB24C9B
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/models/task.js
View file

@ -161,7 +161,7 @@ export default class TaskModel extends AbstractModel {
} }
async cancelScheduledNotifications() { async cancelScheduledNotifications() {
if (!('showTrigger' in Notification.prototype)) { if (!(Notification && 'showTrigger' in Notification.prototype)) {
console.debug('This browser does not support triggered notifications') console.debug('This browser does not support triggered notifications')
return return
} }
@ -196,7 +196,7 @@ export default class TaskModel extends AbstractModel {
return return
} }
if (!('showTrigger' in Notification.prototype)) { if (!(Notification && 'showTrigger' in Notification.prototype)) {
console.debug('This browser does not support triggered notifications') console.debug('This browser does not support triggered notifications')
return return
} }
@ -231,7 +231,7 @@ export default class TaskModel extends AbstractModel {
], ],
}) })
console.debug('Notification scheduled for ' + date) console.debug('Notification scheduled for ' + date)
} catch(e) { } catch (e) {
throw new Error('Error scheduling notification', e) throw new Error('Error scheduling notification', e)
} }
} }