welcome: don't run when settings are absent/updated

Fixes #298

apps.json

@@ -78,7 +78,7 @@
   { "id": "welcome",
     "name": "Welcome",
     "icon": "app.png",
-    "version":"0.06",
+    "version":"0.07",
     "description": "Appears at first boot and explains how to use Bangle.js",
     "tags": "start,welcome",
     "allow_emulator":true,
@@ -86,6 +86,7 @@
       {"name":"welcome.boot.js","url":"boot.js"},
       {"name":"welcome.app.js","url":"app.js"},
       {"name":"welcome.settings.js","url":"settings.js"},
+      {"name":"welcome.settings.json","url":"settings-default.json","evaluate":true},
       {"name":"welcome.img","url":"app-icon.js","evaluate":true}
     ]
   },

apps/welcome/ChangeLog

@@ -4,3 +4,6 @@
 0.04: Fix regression after tweaks to Storage.readJSON
 0.05: Move configuration into App/widget settings
 0.06: Move loader into welcome.boot.js
+0.07: Run again when updated
+      Don't run again when settings app is updated (or absent)
+      Add "Run Now" option to settings

apps/welcome/app.js

@@ -288,6 +288,13 @@ setWatch(()=>{
 }, BTN2, {repeat:true,edge:"rising"});
 setWatch(()=>move(-1), BTN1, {repeat:true});
 
+(function migrateSettings(){
+  let global_settings = require('Storage').readJSON('setting.json', 1)
+  if (global_settings) {
+    delete global_settings.welcomed
+    require('Storage').write('setting.json', global_settings)
+  }
+})()
 
 Bangle.setLCDTimeout(0);
 Bangle.setLCDPower(1);

apps/welcome/boot.js

@@ -1,9 +1,11 @@
 (function() {
-  let s = require('Storage').readJSON('setting.json', 1) || {}
+  let s = require('Storage').readJSON('welcome.settings.json', 1)
+    || require('Storage').readJSON('setting.json', 1)
+    || {welcomed: true} // do NOT run if global settings are also absent
   if (!s.welcomed && require('Storage').read('welcome.app.js')) {
     setTimeout(() => {
       s.welcomed = true
-      require('Storage').write('setting.json', s)
+      require('Storage').write('welcome.settings.json', {welcomed: "yes"})
       load('welcome.app.js')
     })
   }

apps/welcome/settings-default.json

@@ -0,0 +1,3 @@
+{
+  "welcomed": false
+}

apps/welcome/settings.js

@@ -1,16 +1,15 @@
 // The welcome app is special, and gets to use global settings
 (function(back) {
-  let settings = require('Storage').readJSON('setting.json', 1) || {}
+  let settings = require('Storage').readJSON('welcome.settings.json', 1)
+    || require('Storage').readJSON('setting.json', 1) || {}
   E.showMenu({
     '': { 'title': 'Welcome App' },
-    'Run again': {
+    'Run on Next Boot': {
       value: !settings.welcomed,
-      format: v => v ? 'Yes' : 'No',
+      format: v => v ? 'OK' : 'No',
-      onchange: v => {
+      onchange: v => require('Storage').write('welcome.settings.json', {welcomed: !v}),
-        settings.welcomed = v ? undefined : true
-        require('Storage').write('setting.json', settings)
-      },
     },
+    'Run Now': () => load('welcome.app.js'),
     '< Back': back,
   })
 })