search

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

Fix #74 - remove NOS as stream paywalled #76

Closed krisgesling closed 4 years ago

krisgesling commented 4 years ago

As noted in Issue #74 an authentication prompt has been added to the NOS stream.

This simply removes the stream.

forslund commented 4 years ago

Looks good, merging

JarbasAl commented 3 years ago

missed in settingsmeta, still shows up in web ui

krisgesling commented 3 years ago

Yeah I've been holding back changes to settingsmeta as much as possible given it effectively resets all users settings with every tiny change.

Ideally we can handle this gracefully on the backend but if not I think we hold all settings changes and do them in one hit to minimize the number of times users have to re-select their default station.