Skip to content

Commit

Permalink
Merge pull request #143 from debjanibnrj/opendistro-1.4
Browse files Browse the repository at this point in the history
Fix: Security Multitenancy "Show Dashboard" button should redirect users to the dashboards page
  • Loading branch information
debjanibnrj authored Feb 20, 2020
2 parents fd88830 + 181fbba commit 6a03a8d
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions public/apps/multitenancy/multitenancy.js
Original file line number Diff line number Diff line change
Expand Up @@ -205,8 +205,8 @@ uiModules
if (appsToReset.indexOf(navLink.id) > -1) {
chromeWrapper.resetLastSubUrl(navLink.id);
}
});
});

if(chrome.getInjected("timelion.ui.enabled")) {
chromeWrapper.getNavLinkById("timelion").lastSubUrl = chromeWrapper.getNavLinkById("timelion").url;
}
Expand All @@ -227,10 +227,10 @@ uiModules
// redirect to either Visualize or Dashboard depending on user selection.
if(redirect) {
if (redirect == 'vis') {
$window.location.href = chromeWrapper.getNavLinkById("kibana:visualize").url;
$window.location.href = chromeWrapper.getNavLinkById("kibana:visualize").baseUrl;
}
if (redirect == 'dash') {
$window.location.href = chromeWrapper.getNavLinkById("kibana:dashboard").url;
$window.location.href = chromeWrapper.getNavLinkById("kibana:dashboard").baseUrl;
}
} else {
toastNotifications.addSuccess({
Expand Down

0 comments on commit 6a03a8d

Please sign in to comment.