Merge pull request #1 from gregorhollmig/master

Merge fixes for windows file path

Author: avishnyak

src/ScreenHelper.js

@@ -1,23 +1,18 @@
+const { dirname } = require('path');
 class ScreenHelper {
   static async alertError(context, message) {
     return await context.app.alert('Erro!', message);
   }
 
   static async askRepoPath(context, options={}) {
     await context.app.alert(
-      'Selecione Repositorio',
-      `Escolha o repositorio para sincronizar workspaces\nWorkspace Atual: ${options.currentPath}`
+      'Select repository',
+      `Chose the repository to synchronize your workspace\nCurrent path: ${options.currentPath}`
     );
     const path = await context.app.showSaveDialog({defaultPath: options.workspaceName});
 
-    return normalizePath(path);
+    return dirname(path);
   }
 }
 
-const normalizePath = (path) => {
-  if (path == null || path == 'undefined') return null;
-
-  return path.substr(0, path.lastIndexOf('/'));
-};
-
 module.exports = ScreenHelper;

src/index.js

@@ -5,7 +5,7 @@ const ScreenHelper = require('./ScreenHelper.js');
 const verifyRepoConfig = async (repo, context) => {
   if (await repo.isConfigured(context)) return true;
 
-  ScreenHelper.alertError(context, 'Workspace não foi setado!');
+  ScreenHelper.alertError(context, 'Workspace was not set');
   return false;
 };