fix: sequential publish if pwa and github

Author: peterpeterparker

cloud/functions/src/model/data/task.ts

@@ -4,7 +4,7 @@ export interface TaskData {
   deckId: string;
   token: string | firestore.FieldValue;
 
-  type: 'publish-deck' | 'push-github';
+  type: 'publish-all' | 'publish-deck' | 'push-github';
 
   status: 'scheduled' | 'failure' | 'successful';
 

cloud/functions/src/request/publish/schedule-publish-task.ts

@@ -18,7 +18,6 @@ export function schedulePublish(request: functions.Request): Promise<ScheduledPu
     try {
       const token: string | undefined = await geToken(request);
       const deckId: string | undefined = request.body.deckId;
-      const ownerId: string | undefined = request.body.ownerId;
 
       if (!deckId) {
         reject('No deck information provided.');
@@ -30,11 +29,6 @@ export function schedulePublish(request: functions.Request): Promise<ScheduledPu
         return;
       }
 
-      if (!ownerId) {
-        reject('No owner ID provided.');
-        return;
-      }
-
       const publish: boolean = request.body.publish !== undefined && request.body.publish;
       const github: boolean = request.body.github !== undefined && request.body.github;
 
@@ -44,25 +38,10 @@ export function schedulePublish(request: functions.Request): Promise<ScheduledPu
       }
 
       // We tell the frontend to wait
-      await updateDeckDeploy(deckId, ownerId, publish, github);
+      await updateDeckDeploy(deckId, publish, github);
 
       // We schedule internally / cloud the job so we keep secret the token
-
-      if (publish) {
-        await scheduleTask({
-          deckId,
-          token,
-          type: 'publish-deck',
-        });
-      }
-
-      if (github) {
-        await scheduleTask({
-          deckId,
-          token,
-          type: 'push-github',
-        });
-      }
+      await schedule(deckId, publish, github, token);
 
       resolve({
         deckId,
@@ -76,7 +55,35 @@ export function schedulePublish(request: functions.Request): Promise<ScheduledPu
   });
 }
 
-function updateDeckDeploy(deckId: string, ownerId: string, publish: boolean, github: boolean): Promise<void> {
+async function schedule(deckId: string, publish: boolean, github: boolean, token: string) {
+  if (publish && github) {
+    await scheduleTask({
+      deckId,
+      token,
+      type: 'publish-all',
+    });
+
+    return;
+  }
+
+  if (publish) {
+    await scheduleTask({
+      deckId,
+      token,
+      type: 'publish-deck',
+    });
+  }
+
+  if (github) {
+    await scheduleTask({
+      deckId,
+      token,
+      type: 'push-github',
+    });
+  }
+}
+
+function updateDeckDeploy(deckId: string, publish: boolean, github: boolean): Promise<void> {
   return new Promise<void>(async (resolve, reject) => {
     try {
       if (!deckId || deckId === undefined || !deckId) {

cloud/functions/src/watch/publish/publish.ts

@@ -55,13 +55,28 @@ function publishJob(snap: DocumentSnapshot): Promise<void> {
         return;
       }
 
-      if (task.type === 'publish-deck') {
+      if (task.type === 'publish-all') {
+        const newPublish: boolean = deck.data.api_id === undefined || deck.data.api_id === null;
+
+        // If we do both, currently we need the API first as we are getting the content from the published deck
+        await publishToApi(deck, task.token as string);
+
+        // Even if we fixed the delay to publish to Cloudfare CDN (#195), sometimes if too quick, the presentation will not be correctly published
+        // Therefore, to avoid such problem, we add a bit of delay in the process but only for the first publish
+        setTimeout(
+          async () => {
+            await publishToGitHub(deck.id, deck.data);
+            resolve();
+          },
+          newPublish ? 5000 : 0
+        );
+      } else if (task.type === 'publish-deck') {
         await publishToApi(deck, task.token as string);
+        resolve();
       } else if (task.type === 'push-github') {
         await publishToGitHub(deck.id, deck.data);
+        resolve();
       }
-
-      resolve();
     } catch (err) {
       reject(err);
     }