Malawi-DCCMS / weather-app

An app to show the weather.
0 stars 0 forks source link

Search for same location twice leads to error #135

Open LinUXBele opened 3 months ago

LinUXBele commented 3 months ago

BEHAVIOUR When searching, and tapping on the correct location the app will show a spinner until the next decade

STEPS TO REPRODUCE

  1. Open the app
  2. Tap search icon
  3. Search for -la-
  4. Choose Umbula
  5. See Today page for Umbula
  6. Retap the search Icon
  7. Write -la-
  8. Choose Umbula again
  9. Observe spinner spinning forever
  10. Tap anything and the only thing now available is the About us page

EXPECTED BEHAVIOUR When searching nomatter how many times, it should always work ALSO please don't let the Spinner spin forever, after a given amount of Seconds, please throw a error message saying: "Error detected while fetching forecast, please try again" or similar

vegardb commented 3 months ago

A guess of root reason:

If search page takes you back to the same forecast that you had before, the UI only partly updates. It starts waiting for a new forecast, but does not actually request a new one.

vegardb commented 3 months ago

There were two similar issues here:

LinUXBele commented 2 months ago

Tested OK!