new app:bmoface

[LucienStorm]

Hopefully I've cleared all of the issues.

[thyttan]

@bobrippling could you take a look at the checker output? Something is going wrong:

TypeError: app.tags.split is not a function
    at /home/runner/work/BangleApps/BangleApps/bin/sanitycheck.js:209:37
    at Array.forEach (<anonymous>)
    at Object.<anonymous> (/home/runner/work/BangleApps/BangleApps/bin/sanitycheck.js:197:6)
    at Module._compile (node:internal/modules/cjs/loader:1364:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1422:10)
    at Module.load (node:internal/modules/cjs/loader:1203:32)
    at Module._load (node:internal/modules/cjs/loader:1019:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:128:12)
    at node:internal/main/run_main_module:28:49

[RKBoss6]

Ah, I think in metadata.json tags: is not an array, just a string, eg: "clock". The checker doesn't know how to split the array, since it should be a string. Hope that helps!

[LucienStorm]

Metadata file updated

[LucienStorm]

Settings, metadata, and app-icon all updated

[bobrippling]

Yeah that was it - LGTM but will let you finish your review if you like @thyttan

[thyttan]

You're close now 😉 : https://github.com/espruino/BangleApps/actions/runs/18608508358/job/53074607304#step:5:8

So add icon for the app loader and launchers and we're probably good to go.

[LucienStorm]

I've tried different settings on the app icon image compression. Hopefully this passes

[RKBoss6]

You'll also need an app.png file with the icon of the app, not ported to JS. The js icon is for the watch to display it in launchers, and the .png icon is for the app loader. So adding a app.png file with the png version of the app icon in app-icon.js will make this good to go!

[LucienStorm]

[LucienStorm]

I'm going to cancel this pull request and start another with that file. I'm not sure if there's any other way to add a file to a current commit.