Merge pull request #622 from ueokande/fallback-default-settings

Handle errors on loading settings

1 files changed, 21 insertions, 10 deletions

diff --git a/src/background/presenters/NotifyPresenter.ts b/src/background/presenters/NotifyPresenter.ts
index 8fa4acb..9785278 100644
--- a/src/background/presenters/NotifyPresenter.ts
+++ b/src/background/presenters/NotifyPresenter.ts
@@ -1,26 +1,37 @@
 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()
 export default class NotifyPresenter {
-  async notify(
-    title: string,
-    message: string,
-    onclick: () => void,
-  ): Promise<void> {
+  async notifyUpdated(version: string, onclick: () => void): Promise<void> {
+    let title = `Vim Vixen ${version} has been installed`;
+    let message = 'Click here to see release notes';
+
     const listener = (id: string) => {
-      if (id !== NOTIFICATION_ID) {
+      if (id !== NOTIFICATION_ID_UPDATE) {
         return;
       }
-
       onclick();
-
       browser.notifications.onClicked.removeListener(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',
       'iconUrl': browser.extension.getURL('resources/icon_48x48.png'),
       title,