search

snipsco / snips-issues

Feel free to share your bugs with us.
14 stars 5 forks source link

[Feature Request][SAM] update-assistant require to configure skill each time #44

Open oguh22 opened 6 years ago

oguh22 commented 6 years ago

Hello,

I have some skills installed, for example skill-owm (weather) As soon as I do a "sam update-assistant" I need to provide again the API key and default location Would be nice if it can ask for config only the first time. then next time, do not ask for it. we know that if we want to edit the config we need to go to /var/lib/snips/skill ...

bors-ltd commented 5 years ago

Yes, please. This is annoying to press Enter a dozen times each time I update the assistant. I tend to keep the Enter key pressed until I get back to the shell prompt.

This is a developer toolkit, we should know how to ssh.

Maybe even config.ini should not be created by setup.sh but when deploying, using the found template like config.ini.default, asking the user to confirm the values and write it down.

LLUWE19 commented 5 years ago

I think this might be similar problem? Every time I sam install assistant it clears the section of the configuration.yaml for intent_scripts. It creates a backup each time but how do I access it? At the moment I have to save the configuration.yaml somewhere else, then copy and paste after the install is done. Is there a way to avoid this? Or a better practice for doing it.

Thanks