vikunja/frontend
Archived
6
39
Fork 48
Code Issues 1 Pull Requests 4 Projects Releases 26 Activity

fix: only warn once if triggeredNotifications are not supported #2344

Merged
konrad merged 1 commits from dpschen/frontend:feature/fix-triggered-notifications-warning into main 2022-09-14 12:45:22 +00:00
Conversation 4 Commits 1 Files Changed 1 +5 -3
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/models/task.ts
View File

@ -21,9 +21,13 @@ import UserModel from './user'
import AttachmentModel from './attachment' import AttachmentModel from './attachment'
import SubscriptionModel from './subscription' import SubscriptionModel from './subscription'
const SUPPORTS_TRIGGERED_NOTIFICATION = 'Notification' in window && 'showTrigger' in Notification.prototype
export const TASK_DEFAULT_COLOR = '#1973ff' export const TASK_DEFAULT_COLOR = '#1973ff'
const SUPPORTS_TRIGGERED_NOTIFICATION = 'Notification' in window && 'showTrigger' in Notification.prototype
if (!SUPPORTS_TRIGGERED_NOTIFICATION) {
console.debug('This browser does not support triggered notifications')
}
export function getHexColor(hexColor: string) { export function getHexColor(hexColor: string) {
if (hexColor === '' || hexColor === '#') { if (hexColor === '' || hexColor === '#') {
return TASK_DEFAULT_COLOR return TASK_DEFAULT_COLOR
@ -176,7 +180,6 @@ export default class TaskModel extends AbstractModel implements ITask {
async cancelScheduledNotifications() { async cancelScheduledNotifications() {
if (!SUPPORTS_TRIGGERED_NOTIFICATION) { if (!SUPPORTS_TRIGGERED_NOTIFICATION) {
console.debug('This browser does not support triggered notifications')
return return
} }
@ -211,7 +214,6 @@ export default class TaskModel extends AbstractModel implements ITask {
} }
if (!SUPPORTS_TRIGGERED_NOTIFICATION) { if (!SUPPORTS_TRIGGERED_NOTIFICATION) {
console.debug('This browser does not support triggered notifications')
return return
} }