MycroftAI / skill-npr-news

Mycroft AI official News Skill, providing the latest news report from your favorite broadcast.
https://mycroft.ai/skills
Apache License 2.0
9 stars 42 forks source link

catalonia news feed #102

Closed JarbasAl closed 3 years ago

JarbasAl commented 3 years ago

adds catalonia news feed

adds fixes/workaround for #101 #100 #99

curl needs to be disabled in websettings

usage: image

krisgesling commented 3 years ago

Great updates, thanks!

The piece that I'm cautious of is updating the Skill settings. This is an unfortunate limitation of the current backend that needs to be fixed. Every change to settingsmeta will be considered a new settings block and all users settings will be effectively reset. So I've been holding back on changing the settingsmeta until we can either resolve that, or do one big settings update rather than lots of little ones.

For this reason, I'd also prefer not to add the curl setting presuming we'll be removing it in the near future?

I've also been considering expanding the default station functionality to have the option to be more granular. Eg allow state / province based defaults, falling back to a national default, then the global default if neither of the others exist.