Merge pull request #638 from ueokande/qa-0.24

QA 0.24

1 files changed, 10 insertions, 1 deletions

diff --git a/src/background/presenters/NotifyPresenter.ts b/src/background/presenters/NotifyPresenter.ts
index 9785278..defb601 100644
--- a/src/background/presenters/NotifyPresenter.ts
+++ b/src/background/presenters/NotifyPresenter.ts
@@ -26,11 +26,20 @@ export default class NotifyPresenter {
     });
   }
 
-  async notifyInvalidSettings(): Promise<void> {
+  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';
 
+    const listener = (id: string) => {
+      if (id !== NOTIFICATION_ID_INVALID_SETTINGS) {
+        return;
+      }
+      onclick();
+      browser.notifications.onClicked.removeListener(listener);
+    };
+    browser.notifications.onClicked.addListener(listener);
+
     await browser.notifications.create(NOTIFICATION_ID_INVALID_SETTINGS, {
       'type': 'basic',
       'iconUrl': browser.extension.getURL('resources/icon_48x48.png'),