aaron/mattermost-desktop
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

[MM-59480] Remove logging of the title/body of notifications (#3092)

Browse source
This commit is contained in:
Devin Binnie 2024-07-15 14:31:14 -04:00 committed by GitHub
parent 19169b7e70
commit 35665dee33
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/main/app/intercom.ts
View file

@ -113,7 +113,7 @@ export function handleWelcomeScreenModal() {
}
export function handleMentionNotification(event: IpcMainInvokeEvent, title: string, body: string, channelId: string, teamId: string, url: string, silent: boolean, soundName: string) {
log.debug('handleMentionNotification', {title, body, channelId, teamId, url, silent, soundName});
log.debug('handleMentionNotification', {channelId, teamId, url, silent, soundName});
return NotificationManager.displayMention(title, body, channelId, teamId, url, silent, event.sender, soundName);
}

2
src/main/notifications/index.ts
View file

@ -32,7 +32,7 @@ class NotificationManager {
}
public async displayMention(title: string, body: string, channelId: string, teamId: string, url: string, silent: boolean, webcontents: Electron.WebContents, soundName: string) {
log.debug('displayMention', {title, channelId, teamId, url, silent, soundName});
log.debug('displayMention', {channelId, teamId, url, silent, soundName});
if (!Notification.isSupported()) {
log.error('notification not supported');