Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: #12736, merged notifications use username instead of displayname

Browse Source
This commit is contained in:
Julian Lam
2024-08-22 14:55:27 -04:00
parent fbd09b2a4f
commit ad68809bb4
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/notifications.js
View File

@@ -475,7 +475,7 @@ Notifications.merge = async function (notifications) {
case 'notifications:user-posted-to': case 'notifications:user-posted-to':
case 'notifications:user-flagged-post-in': case 'notifications:user-flagged-post-in':
case 'notifications:user-flagged-user': { case 'notifications:user-flagged-user': {
const usernames = _.uniq(set.map(notifObj => notifObj && notifObj.user && notifObj.user.username)); const usernames = _.uniq(set.map(notifObj => notifObj && notifObj.user && notifObj.user.displayname));
const numUsers = usernames.length; const numUsers = usernames.length;
const title = utils.decodeHTMLEntities(notifications[modifyIndex].topicTitle || ''); const title = utils.decodeHTMLEntities(notifications[modifyIndex].topicTitle || '');