openhab / openhab-mycroft

Mycroft skill for openHAB
Eclipse Public License 2.0
20 stars 45 forks source link

Improve initial settings fetch #82

Closed krisgesling closed 4 years ago

krisgesling commented 4 years ago

Hi there, was just giving the Skill a test run as part of the review process and thought we could improve the initial load of settings.

This moves the load of settings to after the Skill has been constructed as web settings will never be available during __init__.

Added a new dialog for configuration needed. Previously it reported that it could not load the list which might be interpreted as - the Skill has connected but something went wrong.