Distinct notification IDs

jh-changes
Shin'ya Ueoka 5 years ago
parent 33a16b85e4
commit 9b2b8f0608
  1. 36
      src/background/presenters/NotifyPresenter.ts
  2. 2
      src/background/usecases/SettingUseCase.ts

@ -1,39 +1,37 @@
import { injectable } from 'tsyringe'; import { injectable } from 'tsyringe';
const NOTIFICATION_ID = 'vimvixen-update'; const NOTIFICATION_ID_UPDATE = 'vimvixen-update';
const NOTIFICATION_ID_INVALID_SETTINGS = 'vimvixen-update-invalid-settings';
@injectable() @injectable()
export default class NotifyPresenter { export default class NotifyPresenter {
async notifyUpdated(version: string, onclick: () => void): Promise<void> { async notifyUpdated(version: string, onclick: () => void): Promise<void> {
let title = `Vim Vixen ${version} has been installed`; let title = `Vim Vixen ${version} has been installed`;
let message = 'Click here to see release notes'; let message = 'Click here to see release notes';
await this.notify(title, message, onclick);
}
async notifyInvalidSettings(onclick: () => void): Promise<void> {
let title = `Loaded settings is invalid`;
// eslint-disable-next-line max-len
let message = 'The default settings is used due to the last saved settings is invalid. Check your current settings from the add-on preference';
await this.notify(title, message, onclick);
}
private async notify(
title: string,
message: string,
onclick: () => void,
): Promise<void> {
const listener = (id: string) => { const listener = (id: string) => {
if (id !== NOTIFICATION_ID) { if (id !== NOTIFICATION_ID_UPDATE) {
return; return;
} }
onclick(); onclick();
browser.notifications.onClicked.removeListener(listener); browser.notifications.onClicked.removeListener(listener);
}; };
browser.notifications.onClicked.addListener(listener); browser.notifications.onClicked.addListener(listener);
await browser.notifications.create(NOTIFICATION_ID, { await browser.notifications.create(NOTIFICATION_ID_UPDATE, {
'type': 'basic',
'iconUrl': browser.extension.getURL('resources/icon_48x48.png'),
title,
message,
});
}
async notifyInvalidSettings(): Promise<void> {
let title = `Loaded settings is invalid`;
// eslint-disable-next-line max-len
let message = 'The default settings is used due to the last saved settings is invalid. Check your current settings from the add-on preference';
await browser.notifications.create(NOTIFICATION_ID_INVALID_SETTINGS, {
'type': 'basic', 'type': 'basic',
'iconUrl': browser.extension.getURL('resources/icon_48x48.png'), 'iconUrl': browser.extension.getURL('resources/icon_48x48.png'),
title, title,

@ -30,7 +30,7 @@ export default class SettingUseCase {
try { try {
value = data.toSettings(); value = data.toSettings();
} catch (e) { } catch (e) {
this.notifyPresenter.notifyInvalidSettings(() => {}); this.notifyPresenter.notifyInvalidSettings();
value = DefaultSettingData.toSettings(); value = DefaultSettingData.toSettings();
} }
this.settingRepository.update(value!!); this.settingRepository.update(value!!);