Merge pull request #4314 from xhon-pelushi/fix/3853-tally-empty-state-to-settings

fix(tally): jump to settings when no tally types are configured

Author: bobrippling

apps/tally/ChangeLog

@@ -1,2 +1,3 @@
 0.01: New app!
 0.02: Handle duplicate menu entries in the app
+0.03: Go straight to settings when no tally types are configured yet, instead of an empty list

apps/tally/app.js

@@ -103,4 +103,10 @@ function tfmt(tally) {
     return "".concat(d.getHours(), ":").concat(pad2(d.getMinutes()));
 }
 var pad2 = function (s) { return ("0" + s.toFixed(0)).slice(-2); };
-showTallies(readTallies());
+var tallyCfg = storage.readJSON("tallycfg.json", 1) || [];
+if (tallyCfg.length === 0) {
+    eval(storage.read("tally.settings.js"))(function () { return load(); });
+}
+else {
+    showTallies(readTallies());
+}

apps/tally/app.ts

@@ -123,4 +123,11 @@ function tfmt(tally: Tally) {
 
 const pad2 = (s: number) => ("0" + s.toFixed(0)).slice(-2);
 
-showTallies(readTallies());
+const tallyCfg = storage.readJSON("tallycfg.json", 1) as TallySettings || [];
+if (tallyCfg.length === 0) {
+  // No tally types configured yet, so there's nothing to log and nothing
+  // to show here - go straight to settings so the user can add one.
+  eval(storage.read("tally.settings.js") as string)(() => load());
+} else {
+  showTallies(readTallies());
+}

apps/tally/metadata.json

@@ -2,7 +2,7 @@
   "id": "tally",
   "name": "Tally Keeper",
   "shortName": "Tally",
-  "version": "0.02",
+  "version": "0.03",
   "author": "bobrippling",
   "description": "Add to a tally from clock info",
   "icon": "app.png",