Run form conversions in series

medic · Jun 1, 2017 · 4160d81 · 4160d81
1 parent 857cef0
commit 4160d81
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "medic-configurer-beta",
-  "version": "1.0.26",
+  "version": "1.0.28",
   "description": "Configure Medic Mobile deployments",
   "main": "index.js",
   "scripts": {

diff --git a/src/lib/convert-forms.js b/src/lib/convert-forms.js
@@ -7,19 +7,19 @@ const trace = require('../lib/log').trace;
 module.exports = (project, subDirectory) => {
   const formsDir = `${project}/forms/${subDirectory}`;
 
-  return Promise.all(
-    fs.readdir(formsDir)
-      .filter(name => name.endsWith('.xlsx'))
-      .map(xls => {
+  return fs.readdir(formsDir)
+    .filter(name => name.endsWith('.xlsx'))
+    .reduce((promiseChain, xls) => {
         const sourcePath = `${formsDir}/${xls}`;
         const targetPath = `${formsDir}/${fs.withoutExtension(xls)}.xml`;
 
-        return Promise.resolve()
+        return promiseChain
           .then(() => info('Converting form', sourcePath, '…'))
           .then(() => xls2xform(sourcePath, targetPath))
           .then(() => fixXml(targetPath))
           .then(() => trace('Converted form', sourcePath));
-      }));
+      },
+      Promise.resolve());
 };
 
 const xls2xform = (sourcePath, targetPath) =>