search

silverbucket / dogfeed

an unhosted feed reader
https://dogfeed.5apps.com
GNU Affero General Public License v3.0
34 stars 4 forks source link

it allows adding invalid feeds. they spin forever. #23

Closed michielbdejong closed 10 years ago

michielbdejong commented 10 years ago

for instance if you fill in 'foo' as the url, i would expect a "that is not a valid url" warning, and if you fill in 'http://example.com/not.a.feed.jpg', i would expect a 'feed not parseable' warning or similar

silverbucket commented 10 years ago

hmm, what should happen right now is it's added with a redish-pink background indicating it's broken. then you can delete it.

i will try to put some up-front error checking before the feed is stored/represented in the list. but the initial problem you describe (the spinning) should be fixed, it was probably due to the add feed error.

silverbucket commented 10 years ago

please re-submit if you're getting 'spin forever' .. i mentioned that should be fixed. anyway. made a separate bug just for the 'url validation'

https://github.com/silverbucket/dogfeed/issues/31