MycroftAI/skill-configuration

Be more descriptive on configuration update outcome

Opened this issue · 1 comments

It was reported in Chat that Mycroft had responded that it didn't have any configuration updates, yet Skill settings had indeed changed.

The current config update intent is setup in this way. The settings update is triggered, then a configuration update. However the response is only referring to whether the config got updated, it has no idea whether any Skill settings were actually updated or not.

Maybe we need to be more descriptive in the response, particularly because the end user may not see a difference between a configuration (device) and settings (Skills). We do also refer to configurations as "device settings" in various places. Perhaps something like:

I'm syncing your Skill settings.
Your device settings have also been updated.

OR

I'm syncing your Skill settings.
I couldn't find any changes for your device settings.

Also just saw that this intent doesn't have any Voight Kampff tests. In fixing the above, we should also add one or more tests.