Skip to content
This repository has been archived by the owner on Jul 17, 2020. It is now read-only.

feature request #90

Open
ghost opened this issue Apr 11, 2019 · 1 comment
Open

feature request #90

ghost opened this issue Apr 11, 2019 · 1 comment

Comments

@ghost
Copy link

ghost commented Apr 11, 2019

can we get MqttOptions a method to load its field values from snips.toml file

there would be cases for hermes-python to not load values, so that users can enter in values themselves...
but for snips-skill-server app skills, the MqttOptions should be using mqtt config set within the snips.toml file

We should not have to include all mqtt settings within every snips-skill-server app when this information is already setup in the snips.toml
I should be able to change a single item in the snips.toml and that's it
I should not have to change not only the snips.toml file but then also EVERY config.ini within every single skill folder also

snips.toml is the mecca for settings regarding my snips setup, and ALL snips programs should be using this as the basis for information, or at least provide a single method call on objects like MqttOptions to be able to know i want it to load information from there with me having to manually do it

@maxbachmann
Copy link

I would love this especially because quite a lot of skills use hermes, but do ignore the settings from the snips.toml

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant