Skip to content

Commit e6abef0

Browse files
authored
Merge pull request #1848 from akhilkr128/notification_bug
Fix for missing text in Notifications drawer
2 parents 76923bb + 862e035 commit e6abef0

File tree

6 files changed

+5
-13
lines changed

6 files changed

+5
-13
lines changed

client/app/components/notifications/subheading.html

Lines changed: 0 additions & 1 deletion
This file was deleted.

client/app/core/core.module.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ import { ExceptionModule } from './exception/exception.module.js'
2323
import { LanguageFactory } from './language.service.js'
2424
import { ListConfigurationFactory } from './list-configuration.service.js'
2525
import { ListViewFactory } from './list-view.service.js'
26-
import { NavigationController, navigationInit } from './navigation/navigation-controller.js'
26+
import { NavigationController } from './navigation/navigation-controller.js'
2727
import { NavigationFactory } from './navigation.service.js'
2828
import { PollingFactory } from './polling.service.js'
2929
import { RBACFactory } from './rbac.service.js'
@@ -93,5 +93,4 @@ export const CoreModule = angular
9393
.run(authInit)
9494
.run(gettextInit)
9595
.run(layoutInit)
96-
.run(navigationInit)
9796
.name

client/app/core/navigation/navigation-controller.js

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,7 @@ export function NavigationController (Text, Navigation, Session, API_BASE, Shopp
6060
helpMenu: RBAC.hasAny(['about', 'product', 'documentation'])
6161
}
6262
EventNotifications.setToastDisplay(vm.permissions.suiNotifications)
63+
const appBasePath = process.env.NODE_ENV === 'production' ? '' : 'assets/'
6364
angular.extend(vm, {
6465
state: Navigation.state,
6566
text: Text.app,
@@ -71,10 +72,9 @@ export function NavigationController (Text, Navigation, Session, API_BASE, Shopp
7172
notificationsDrawerShown: false,
7273
newNotifications: false,
7374
html: {
74-
// injected into templateCache by navigationInit
75-
heading: 'notifications/heading.html',
76-
notificationBody: 'notifications/notification-body.html',
77-
notificationFooter: 'notifications/notification-footer.html',
75+
heading: `${appBasePath}html/heading.html`,
76+
notificationBody: `${appBasePath}html/notification-body.html`,
77+
notificationFooter: `${appBasePath}html/notification-footer.html`,
7878
},
7979
handleItemClick: handleItemClick,
8080
toggleNotificationsList: () => { vm.notificationsDrawerShown = !vm.notificationsDrawerShown },
@@ -250,9 +250,3 @@ export function NavigationController (Text, Navigation, Session, API_BASE, Shopp
250250
}
251251
}
252252

253-
/** @ngInject */
254-
export const navigationInit = function($templateCache) {
255-
$templateCache.put('notifications/heading.html', require('../../components/notifications/heading.html'));
256-
$templateCache.put('notifications/notification-body.html', require('../../components/notifications/notification-body.html'));
257-
$templateCache.put('notifications/notification-footer.html', require('../../components/notifications/notification-footer.html'));
258-
};

0 commit comments

Comments
 (0)