nikdo / wind-alert

Alerts wind freaks about big days.
0 stars 0 forks source link

Browser navigation does not work #103

Closed nikdo closed 4 years ago

nikdo commented 4 years ago

Steps to reproduce:

  1. Navigate to few spots after each other.
  2. Use back button.
  3. Use forward button.

Current behavior: Spot is not changed. Browser history reacts only to home page ↔ detail page change.

nikdo commented 4 years ago

Findings

There is nothing wrong with react-router. There is an incorrect lifecycle method and condition for calling fetchSpotDetail from SpotDetail container. It should react to ID change too, but it should not do history.push in that case.

Tasks

Time estimated: 6h Time spent: 4h