fix(Menu): Add fallback to service menu when service name is empty

Closes #250
meetfranz · Nov 18, 2017 · 42ed24d · 42ed24d
1 parent 8736c15
commit 42ed24d
Showing 1 changed file with 17 additions and 1 deletion.
diff --git a/src/lib/Menu.js b/src/lib/Menu.js
@@ -277,7 +277,7 @@ export default class FranzMenu {
 
     if (this.stores.user.isLoggedIn) {
       return services.map((service, i) => ({
-        label: service.name,
+        label: this._getServiceName(service),
         accelerator: i <= 9 ? `CmdOrCtrl+${i + 1}` : null,
         type: 'radio',
         checked: service.isActive,
@@ -289,4 +289,20 @@ export default class FranzMenu {
 
     return [];
   }
+
+  _getServiceName(service) {
+    if (service.name) {
+      return service.name;
+    }
+
+    let name = service.recipe.name;
+
+    if (service.team) {
+      name = `${name} (${service.team})`;
+    } else if (service.customUrl) {
+      name = `${name} (${service.customUrl})`;
+    }
+
+    return name;
+  }
 }