-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added Utils.readStorageJSON with relaxed JSON parser #3110
Conversation
…erface.html over to using it. This allows Bangle.js 2v20 (or cutting edge) and later to store a relaxed 'JSON' on internal storage which (while still normal JS) is smaller and faster (and preserves unicode better) See espruino/Espruino#2429
tested read good on calendar, grocery, noteify, openstmap, sleepphasealarm, mylocation, sched, also with special character "\xFC" working |
Co-authored-by: Rob Pilling <[email protected]>
Co-authored-by: Rob Pilling <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks grand, shall leave to you to merge @gfwilliams in case you're wanting to time it with any other changes
Thanks! I won't push this to banglejs.com/apps until next week just in case ;) |
Added Utils.readStorageJSON and relaxed JSON parser, and swap all interface.html over to using it.
This allows Bangle.js 2v20 (or cutting edge) and later to store a relaxed 'JSON' on internal storage which (while still normal JS) is smaller and faster (and preserves unicode better)
Fixes espruino/Espruino#2429
It'd be great if you could take a look over this as I had to change a bunch of files. I've tried some (eg openstmap) and they work ok, but it may well be I broke something :(