Make API.read wait on all requests on the SequentialQueue to respond

src/libs/API.js

@@ -105,7 +105,7 @@ function makeRequestWithSideEffects(command, apiCommandParameters = {}, onyxData
  * @param {Object} [onyxData.failureData] - Onyx instructions that will be passed to Onyx.update() when the response has jsonCode !== 200.
  */
 function read(command, apiCommandParameters, onyxData) {
-    makeRequestWithSideEffects(command, apiCommandParameters, onyxData, CONST.API_REQUEST_TYPE.READ);
+    SequentialQueue.getIsDonePromise().then(() => makeRequestWithSideEffects(command, apiCommandParameters, onyxData, CONST.API_REQUEST_TYPE.READ));
 }
 
 export {

src/libs/Network/SequentialQueue.js

@@ -18,6 +18,13 @@ let isSequentialQueueRunning = false;
 
 let currentRequest = null;
 
+let resolveIsDonePromise;
+let isDonePromise = new Promise((resolve) => {
+    resolveIsDonePromise = resolve;
+});
+
+let pendingRequests = [];
+
 /**
  * This method will get any persisted requests and fire them off in sequence to retry them.
  *
@@ -33,6 +40,7 @@ function process() {
 
     const task = _.reduce(persistedRequests, (previousRequest, request) => previousRequest.then(() => {
         currentRequest = Request.processWithMiddleware(request, true);
+        pendingRequests.push(currentRequest);
         return currentRequest;
     }), Promise.resolve());
 
@@ -68,6 +76,19 @@ function flush() {
                     isSequentialQueueRunning = false;
                     resolveIsReadyPromise();
                     currentRequest = null;
+
+                    // Resolve the isDonePromise once all the pending requests complete,
+                    // and reset the pendingRequests array
+                    Promise.all(pendingRequests).then(() => {
+                        resolveIsDonePromise();
+                        pendingRequests = [];
+                    });
+                }).then(() => {
+                    // Reset the isDonePromise so we can use it again on the next run,
+                    // this has to be done after the finally runs otherwise it'll have no effect
+                    isDonePromise = new Promise((resolve) => {
+                        resolveIsDonePromise = resolve;
+                    });
                 });
         },
     });
@@ -114,9 +135,17 @@ function getCurrentRequest() {
     return currentRequest;
 }
 
+function getIsDonePromise() {
+    if (!isSequentialQueueRunning) {
+        return Promise.resolve();
+    }
+    return isDonePromise;
+}
+
 export {
     flush,
     getCurrentRequest,
     isRunning,
     push,
+    getIsDonePromise,
 };