Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Config page tries to send old removed settings #10

Closed
gregoiresage opened this issue Feb 21, 2016 · 0 comments · Fixed by #82
Closed

Config page tries to send old removed settings #10

gregoiresage opened this issue Feb 21, 2016 · 0 comments · Fixed by #82
Labels
bug low priority
Milestone
v0.1.6

Comments

@gregoiresage
Copy link
Contributor

If I remove a setting from the configuration (config.json), when I press Save on the new config page, 'Clay' tries to send the old setting (cached in the localstorage or something like that)

[14:34:26] javascript> Unknown message key 'toto', you should add it in your appinfo.json/appKeys file, for now it'll be removed from the final message sent.

Priority low :p
@keegan-lillo keegan-lillo modified the milestone: v0.1.7 Apr 14, 2016
@keegan-lillo keegan-lillo mentioned this issue May 29, 2016
Merged
keegan-lillo added a commit that referenced this issue May 29, 2016
@keegan-lillo
Merge pull request #82 from pebble/#10/remove-old-settings
7e2b75d 
Only send settings that are actually in the config back to the watch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug low priority
Projects
None yet
Milestone
v0.1.6
Development

Successfully merging a pull request may close this issue.

Only send settings that are actually in the config back to the watch pebble/clay
2 participants
@keegan-lillo @gregoiresage