plausible/analytics
1
1
Fork 0
mirror of https://github.com/plausible/analytics.git synced 2024-11-28 21:47:25 +03:00
Code Issues Packages Projects Releases Wiki Activity

Make countries/regions/cities tooltips work

Browse Source 
Previously these tooltips said `Add filter: Country is undefined`
because new labels were yet to be in `query`.
This commit is contained in:
Karl-Aksel Puulmann 2024-09-16 09:41:36 +03:00
parent 748e0d902f
commit 875dd368f1
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
CHANGELOG.md
View File

@ -51,6 +51,7 @@ All notable changes to this project will be documented in this file.
- Fix property filter suggestions 500 error when property hasn't been selected - Fix property filter suggestions 500 error when property hasn't been selected
- Bamboo.Mua: add Date and Message-ID headers if missing plausible/analytics#4474 - Bamboo.Mua: add Date and Message-ID headers if missing plausible/analytics#4474
- Fix migration order across `plausible_db` and `plausible_events_db` databases plausible/analytics#4466 - Fix migration order across `plausible_db` and `plausible_events_db` databases plausible/analytics#4466
- Fix tooltips for countries/cities/regions links in dashboard
## v2.1.1 - 2024-06-06 ## v2.1.1 - 2024-06-06

4
assets/js/dashboard/stats/reports/list.js
View File

@ -28,8 +28,8 @@ export function FilterLink({ path, filterInfo, onClick, children, extraClass })
const newLabels = cleanLabels(newFilters, query.labels, filter[1], labels) const newLabels = cleanLabels(newFilters, query.labels, filter[1], labels)
return ( return (
<AppNavigationLink <AppNavigationLink
title={`Add filter: ${plainFilterText(query, filter)}`} title={`Add filter: ${plainFilterText({ ...query, labels: newLabels }, filter)}`}
className={className} className={className}
path={path} path={path}
onClick={onClick} onClick={onClick}