mirror of
https://github.com/discourse/discourse.git
synced 2025-01-18 17:52:45 +08:00
FIX: missing appEvents param for onNotification (#27180)
The `onNotification` signature is: ``` onNotification(data, siteSettings, user, appEvents) ``` And we were not passing `appEvents`. Also explicitly inject `currentUser` in `chat-notification-manager` service.
This commit is contained in:
parent
bc089dc52b
commit
14b8894ddb
|
@ -11,6 +11,8 @@ export default class ChatNotificationManager extends Service {
|
|||
@service presence;
|
||||
@service chat;
|
||||
@service chatStateManager;
|
||||
@service currentUser;
|
||||
@service appEvents;
|
||||
|
||||
_subscribedToCore = true;
|
||||
_subscribedToChat = false;
|
||||
|
@ -143,7 +145,12 @@ export default class ChatNotificationManager extends Service {
|
|||
|
||||
@bind
|
||||
onMessage(data) {
|
||||
return onNotification(data, this.siteSettings, this.currentUser);
|
||||
return onNotification(
|
||||
data,
|
||||
this.siteSettings,
|
||||
this.currentUser,
|
||||
this.appEvents
|
||||
);
|
||||
}
|
||||
|
||||
_shouldRun() {
|
||||
|
|
Loading…
Reference in New Issue
Block a user