fedora-infra / mobile

An Android app that allows the Fedora community to interact with Fedora's web apps on the go.
https://fedoraproject.org/mobile
Other
27 stars 10 forks source link

Fix non-totality of PullToRefreshAttacher in MainActivity #23

Open relrod opened 10 years ago

relrod commented 10 years ago

f71aebe fixes a crash that occurred when HRF or Datagrepper were offline or threw an error. But it creates a new issue in that it is possible to enter a state in which the newsfeed becomes unusable until the app is restarted. We should have a timeout in place and mark the refresh "complete" after a few seconds (and show a toast with an error).

relrod commented 10 years ago

Refs #27. This should be part of that cleanup. Right now this is Promise[String \/ List[HRF.Result]].