search

pgaskin / NickelMenu

The easiest way to launch scripts, change settings, and run actions on Kobo e-readers.
https://pgaskin.net/NickelMenu
MIT License
556 stars 28 forks source link

Fix reloaded config only propagating to new menus or the first existing one #58

Closed pgaskin closed 4 years ago

pgaskin commented 4 years ago

Instead of returning a bool from nm_global_config_update indicating whether the menu items were modified, return an int indicating the current config revision which can then be compared against one stored as a property of the menu.

closes #57

pgaskin commented 4 years ago

@jackiew1, does this fix the issue you reported in #56?

https://nm.storage.geek1011.net/artifacts/commit/df5332bbc71e0f52cfbfd34279bd0ca39dd417dc/KoboRoot.tgz

jackiew1 commented 4 years ago

@jackiew1, does this fix the issue you reported in #56?

https://nm.storage.geek1011.net/artifacts/commit/df5332bbc71e0f52cfbfd34279bd0ca39dd417dc/KoboRoot.tgz

Yes, that seems to have done the trick. I tested using the same sequence of events originally posted in #56 👍