Pārlūkot izejas kodu

fix: workflow trigger not registered

Ahmad Kholid 2 gadi atpakaļ
vecāks
revīzija
7a397cc8e2

+ 1 - 1
src/background/BackgroundEventsListeners.js

@@ -15,7 +15,7 @@ class BackgroundEventsListeners {
     BackgroundWorkflowTriggers.scheduleWorkflow(event);
   }
 
-  static onHistoryStateUpdated({ tabId, url, frameId }) {
+  static onWebNavigationCompleted({ tabId, url, frameId }) {
     if (frameId > 0) return;
 
     BackgroundWorkflowTriggers.visitWebTriggers(tabId, url);

+ 3 - 1
src/background/BackgroundUtils.js

@@ -27,8 +27,10 @@ class BackgroundUtils {
       } else {
         const windowOptions = {
           url: tabUrl,
+          height: 715,
+          width: 750,
           type: 'popup',
-          state: updateTab ? 'maximized' : 'minimized',
+          focused: updateTab,
         };
 
         await browser.windows.create(windowOptions);

+ 8 - 0
src/background/BackgroundWorkflowTriggers.js

@@ -182,6 +182,14 @@ class BackgroundWorkflowTriggers {
         if (isStartup && triggerBlock.type === 'on-startup') {
           BackgroundWorkflowUtils.executeWorkflow(currWorkflow);
         } else {
+          if (isStartup && triggerBlock.triggers) {
+            for (const trigger of triggerBlock.triggers) {
+              if (trigger.type === 'on-startup') {
+                await BackgroundWorkflowUtils.executeWorkflow(currWorkflow);
+              }
+            }
+          }
+
           await registerWorkflowTrigger(currWorkflow.id, {
             data: triggerBlock,
           });

+ 2 - 2
src/background/index.js

@@ -23,8 +23,8 @@ browser.runtime.onInstalled.addListener(
   BackgroundEventsListeners.onRuntimeInstalled
 );
 
-browser.webNavigation.onHistoryStateUpdated.addListener(
-  BackgroundEventsListeners.onHistoryStateUpdated
+browser.webNavigation.onCompleted.addListener(
+  BackgroundEventsListeners.onWebNavigationCompleted
 );
 
 const contextMenu =