Skip to content

Commit

Permalink
Make new tab dashboard links work (plausible#4276)
Browse files Browse the repository at this point in the history
* Make new tab dashboard links work

* Urlencode domain

* Recognize shared links
  • Loading branch information
aerosol authored Jun 25, 2024
1 parent 70fca3e commit fa065c8
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 5 deletions.
3 changes: 2 additions & 1 deletion assets/js/dashboard.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,8 @@ if (container) {
background: container.dataset.background,
isDbip: container.dataset.isDbip === 'true',
flags: JSON.parse(container.dataset.flags),
validIntervalsByPeriod: JSON.parse(container.dataset.validIntervalsByPeriod)
validIntervalsByPeriod: JSON.parse(container.dataset.validIntervalsByPeriod),
shared: !!container.dataset.sharedLinkAuth,
}

const loggedIn = container.dataset.loggedIn === 'true'
Expand Down
9 changes: 5 additions & 4 deletions assets/js/dashboard/router.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,10 @@ function ScrollToTop() {
}

export default function Router({ site, loggedIn, currentUserRole }) {
console.info('xxx', site);
return (
<BrowserRouter>
<Route path="/:domain">
<BrowserRouter basename={site.shared ? `/share/${encodeURI(site.domain)}` : encodeURI(site.domain)}>
<Route path="/">
<ScrollToTop />
<Dash site={site} loggedIn={loggedIn} currentUserRole={currentUserRole} />
<Switch>
Expand All @@ -52,7 +53,7 @@ export default function Router({ site, loggedIn, currentUserRole }) {
<ExitPagesModal site={site} />
</Route>
<Route path="/:domain/countries">
<ModalTable title="Top countries" site={site} endpoint={url.apiPath(site, '/countries')} filterKey="country" keyLabel="Country" renderIcon={renderCountryIcon} showPercentage={true}/>
<ModalTable title="Top countries" site={site} endpoint={url.apiPath(site, '/countries')} filterKey="country" keyLabel="Country" renderIcon={renderCountryIcon} showPercentage={true} />
</Route>
<Route path="/:domain/regions">
<ModalTable title="Top regions" site={site} endpoint={url.apiPath(site, '/regions')} filterKey="region" keyLabel="Region" renderIcon={renderRegionIcon} />
Expand All @@ -71,7 +72,7 @@ export default function Router({ site, loggedIn, currentUserRole }) {
</Route>
</Switch>
</Route>
</BrowserRouter>
</BrowserRouter >
);
}

Expand Down

0 comments on commit fa065c8

Please sign in to comment.