outbreak-info / outbreak.info

During outbreaks of emerging diseases such as COVID-19, efficiently collecting, sharing, and integrating data is critical to scientific research. outbreak.info is a resource to aggregate all this information into a single location.
https://outbreak.info/
GNU General Public License v3.0
33 stars 13 forks source link

Routing from the Location page to Situation Reports pages does not work #641

Closed flaneuse closed 1 year ago

flaneuse commented 1 year ago

If you click on the following places, you should link to /situation-reports pages with various parameters. However, instead it routes to the Situation Report home page:

  1. Routing from stream graph

    Screenshot 2023-04-03 at 2 59 27 PM
  2. Routing from stacked bar chart

    Screenshot 2023-04-03 at 2 59 37 PM
  3. heatmap (clicking on an axis label or cell)

Screenshot 2023-04-03 at 2 59 44 PM
  1. lineages over time

    Screenshot 2023-04-03 at 2 58 54 PM
  2. table

    Screenshot 2023-04-03 at 2 59 56 PM
flaneuse commented 1 year ago

Also broken from any of the links on http://localhost:8080/situation-reports or if you directly load http://localhost:8080/situation-reports?pango=BA.5

(all redirect to http://localhost:8080/situation-reports)

Possibly related to #650

olekkorob commented 1 year ago

Fixed

leandrocollares commented 1 year ago

Routing from the Tracked lineages table can yield the this.$route.query.pango.toUpperCase is not a function error.

Could you please add this.$route.query.pango ? this.$route.query.pango.toUpperCase : null and check if the routing takes place without errors?

olekkorob commented 1 year ago

As you can see, I have added it already. It doesn't solve a problem. So I implemented in other way. Please check again.

olekkorob commented 1 year ago

Fixed.