samuelclay / NewsBlur

NewsBlur is a personal news reader that brings people together to talk about the world. A new sound of an old instrument.
http://www.newsblur.com
MIT License
6.92k stars 1k forks source link

Android: Persistent "Error loading stories" notification #453

Closed joemilbourn closed 10 years ago

joemilbourn commented 10 years ago

Hitting refresh in either the main screen titled "news blur" or the "all stories" screen gives a popup "Error loading stories" notification and an odd progress indicator. The error loading stories notification stays on returning to the android home screen, after stopping newsblur, until the device is rebooted.

Version from play store, 5th Jan '14, updated 6th Jan '14; reproduced on Nexus 7 (Kitkat), Moto G (Jellybean).

imd commented 10 years ago

I have the same problem, using a Nexus 7 2013 with KitKat, and NewsBlur 3.5.

dosiecki commented 10 years ago

Thanks for the report. We have been chasing this bug, but need more info to reproduce. Anything helps: app settings, ROM info, network info, other apps in use, etc.

dosiecki commented 10 years ago

The 3.5.1 update should at least make it stop when you exit the app. Still hunting down the root cause.

joemilbourn commented 10 years ago

Hi Daniel,

Seems independent of other apps - occurs just after boot when running only newsblur.

I don't have the nexus to hand now, but the Moto G is running android 4.3, kernel version 3.4.0-g7161c0 and system version 14.71.8.falcon_umts en.GB

If it helps isolate network problems, my nexus doesn't have modem and exhibits the same problem.

Can I export the newsblur settings for you to examine? Otherwise, I have:

HTH, and thanks,

Joe

On 6 Jan 14, Daniel said:

Thanks for the report. We have been chasing this bug, but need more info to reproduce. Anything helps: app settings, ROM info, network info, other apps in use, etc.


Reply to this email directly or view it on GitHub: https://github.com/samuelclay/NewsBlur/issues/453#issuecomment-31669528

This is all because respectable scientists don't get invited to “those kinds of parties.”

joemilbourn commented 10 years ago

On 6 Jan 14, Daniel said:

The 3.5.1 update should at least make it stop when you exit the app. Still hunting down the root cause.

Thanks, that's helped.

Joe

Coffee has become recognized as a human necessity. It is no longer a luxury or an indulgence; it is a corollary of human energy and human efficiency.

seh commented 10 years ago

I experienced this same problem this morning on a Moto X with Newsblur version 3.5. The only other app running was Email. It took rebooting the phone to get the "Error loading stories" notification bubble to go away.

dosiecki commented 10 years ago

As of 3.5.1 the message should only show a single time, and only if there was an actual error encountered. We still haven't tracked down the source of the errors, so if anyone tech-savvy (preferably running 3.5.5 or higher) sends along a log, it would be a great help!

joemilbourn commented 10 years ago

The latest update (8th Jan '14) has fixed the odd progress indicator, and shows only one error loading stories notification.

If you can point me to some instructions, I'll have a go at capturing a log.

Cheers, Joe

dosiecki commented 10 years ago

https://getsatisfaction.com/newsblur/topics/persistent_error_loading_stories_overlay_on_screen

Finally got a break on this! Was able to repro using a non-premium account. Fix in progress.

dosiecki commented 10 years ago

Looks like for non-premium users, the API returns both a valid response and an error field. We recently started handling errors, and the dual case wasn't correctly written. PR incoming.

samuelclay commented 10 years ago

Drat, I think I know the parameter you're talking about. I can see myself agreeing to that sort of inconsistent behavior for some important endpoint like the single feed's stories endpoint. Anyway, this should probably be fixed on the client/Android side, since somebody somewhere is relying on that error parameter being filled for some other client. Ugh, I wish I had it in me to rewrite the entire API so it could get OAuth and more consistency.

dosiecki commented 10 years ago

Fixed in #462.