Updated notification to only show if the island is not on the report page already

This commit is contained in:
Shay Nehmad 2019-09-09 11:47:16 +03:00
parent 313911fd77
commit 63d76f19f8
1 changed files with 13 additions and 10 deletions

View File

@ -29,7 +29,7 @@ let infectionMonkeyImage = require('../images/infection-monkey.svg');
let guardicoreLogoImage = require('../images/guardicore-logo.png');
let notificationIcon = require('../images/notification-logo-512x512.png');
const reportZeroTrustPath = '/report/zero_trust';
const reportZeroTrustRoute = '/report/zero_trust';
class AppComponent extends AuthComponent {
updateStatus = () => {
@ -202,7 +202,7 @@ class AppComponent extends AuthComponent {
{this.renderRoute('/infection/telemetry', <TelemetryPage onStatusChange={this.updateStatus}/>)}
{this.renderRoute('/start-over', <StartOverPage onStatusChange={this.updateStatus}/>)}
{this.renderRoute('/report/security', <ReportPage onStatusChange={this.updateStatus}/>)}
{this.renderRoute(reportZeroTrustPath, <ZeroTrustReportPage onStatusChange={this.updateStatus}/>)}
{this.renderRoute(reportZeroTrustRoute, <ZeroTrustReportPage onStatusChange={this.updateStatus}/>)}
{this.renderRoute('/license', <LicensePage onStatusChange={this.updateStatus}/>)}
</Col>
</Row>
@ -213,9 +213,11 @@ class AppComponent extends AuthComponent {
showInfectionDoneNotification() {
if (this.state.completedSteps.infection_done) {
// No need to show the notification to redirect to the report if we're already in the report page
if (!window.location.href.includes("report")) {
const hostname = window.location.hostname;
const port = window.location.port;
let url = `https://${hostname}:${port}${reportZeroTrustPath}`;
let url = `https://${hostname}:${port}${reportZeroTrustRoute}`;
Notifier.start(
"Monkey Island",
@ -225,6 +227,7 @@ class AppComponent extends AuthComponent {
}
}
}
}
AppComponent.defaultProps = {};