AmericanWhitewater / aw-ios

American Whitewater's iOS app
5 stars 2 forks source link

Reach Detail: Lacking a No Connection State #241

Closed Gregliest closed 3 years ago

Gregliest commented 3 years ago

On the reach detail page, there's no indication that data can't be loaded. The screen shows only "no details available", which is the same state as if there was no description.

yeahphil commented 3 years ago

Ah I threw together a commit for this but it's a bit of a mess. Details get loaded via a separate API call then saved locally (in CD) to Reach.

So if the network call fails AND there aren't details stored locally the details controller shows an error message now. Otherwise it shows what it has. Seem reasonable?