fix file getting

Author: SaraVieira

packages/app/src/app/overmind/effects/git/export-to-github.ts

@@ -1,29 +1,23 @@
 import { Sandbox } from '@codesandbox/common/lib/types';
 import JSZip from 'jszip';
-import { createZip, BLOB_ID } from '../zip/create-zip';
+import zip from '../zip';
 
+const BLOB_ID = 'blob-url://';
 export default async function deploy(sandbox: Sandbox) {
   // We first get the zip file, this is what we essentially need to have deployed.
   // So we convert it to an API request that ZEIT will understand
-  const zipFile = await createZip(
-    sandbox,
-    sandbox.modules,
-    sandbox.directories,
-    false,
-    true
-  );
+  const zipFile = await zip.create(sandbox);
 
   if (!zipFile) {
     return null;
   }
 
   const contents = await JSZip.loadAsync(zipFile);
-
   const apiData = { sandbox: [] };
-  const filePaths = Object.keys(contents.files);
+  const filePaths = Object.keys(contents);
   for (let i = 0; i < filePaths.length; i += 1) {
     const filePath = filePaths[i];
-    const file = contents.files[filePath];
+    const file = contents[filePath];
 
     if (!file.dir) {
       let content = await file.async('text'); // eslint-disable-line no-await-in-loop

packages/app/src/app/overmind/effects/zip.ts

@@ -12,18 +12,19 @@ const getFile = async (id: string) => {
     responseType: 'arraybuffer',
   });
 
-  return file.data;
+  const blob = new Blob([file.data], {
+    type: 'application/zip',
+  });
+
+  return blob;
 };
 
 export default {
-  create({ id }, { id: string }) {
+  create({ id }: { id: string }) {
     return getFile(id);
   },
   async download({ title, id }: { title: string; id: string }) {
     const file = await getFile(id);
-    const blob = new Blob([file], {
-      type: 'application/zip',
-    });
-    saveAs(blob, `${title || id}.zip`);
+    saveAs(file, `${title || id}.zip`);
   },
 };

packages/app/src/app/overmind/namespaces/deployment/actions.ts

@@ -24,10 +24,7 @@ export const deployWithNetlify: AsyncAction = async ({
   const zip = await effects.zip.create(state.editor.currentSandbox);
 
   try {
-    const id = await effects.netlify.deploy(
-      zip.file,
-      state.editor.currentSandbox
-    );
+    const id = await effects.netlify.deploy(zip, state.editor.currentSandbox);
     state.deployment.deploying = false;
 
     await actions.deployment.getNetlifyDeploys();
@@ -95,7 +92,7 @@ export const deployClicked: AsyncAction = async ({
   try {
     state.deployment.deploying = true;
     const zip = await effects.zip.create(state.editor.currentSandbox);
-    const contents = await effects.jsZip.loadAsync(zip.file);
+    const contents = await effects.jsZip.loadAsync(zip);
     state.deployment.url = await effects.zeit.deploy(
       contents,
       state.editor.currentSandbox

packages/app/src/app/pages/Dashboard/Content/SandboxGrid/index.tsx

@@ -6,7 +6,7 @@ import track from '@codesandbox/common/lib/utils/analytics';
 import { getSandboxName } from '@codesandbox/common/lib/utils/get-sandbox-name';
 import { protocolAndHost } from '@codesandbox/common/lib/utils/url-generator';
 import { makeTemplates } from 'app/components/CreateNewSandbox/queries';
-import downloadZip from 'app/overmind/effects/zip/create-zip';
+import zip from 'app/overmind/effects/zip';
 import { formatDistanceToNow } from 'date-fns';
 import { zonedTimeToUtc } from 'date-fns-tz';
 import { camelizeKeys } from 'humps';
@@ -231,9 +231,7 @@ class SandboxGridComponent extends React.Component<
     const sandboxes = await Promise.all(
       sandboxIds.map(s => this.getSandbox(s))
     );
-    return Promise.all(
-      sandboxes.map(s => downloadZip(s, s.modules, s.directories))
-    );
+    return Promise.all(sandboxes.map(s => zip.download(s)));
   };
 
   forkSandbox = id => {