Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2026-05-07 04:06:17 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix: merged chat notifications if all the messages are from the same user

Browse Source 
usernames.length ends up being 1 so need to use default translation string and not -dual/-triple/-multiple variants
This commit is contained in:
Barış Soner Uşaklı
2026-03-12 22:02:03 -04:00
parent 4fef1b4f9d
commit 6147a4d082
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/notifications.js
View File

@@ -519,7 +519,9 @@ Notifications.merge = async function (notifications) {
differentiators.forEach((differentiator) => {
function typeFromLength(items) {
if (items.length === 2) {
if (items.length <= 1) {
return '';
} else if (items.length === 2) {
return 'dual';
} else if (items.length === 3) {
return 'triple';
@@ -553,7 +555,7 @@ Notifications.merge = async function (notifications) {
const type = typeFromLength(usernames);
const isMultiple = type === 'multiple';
const txArgs = [
`${mergeId}-${type}`,
`${mergeId}${type ? '-type' : ''}`,
...usernames.slice(0, usernames.length <= 3 ? 3 : 2),
...(isMultiple ? [usernames.length - 2] : []),
notifObj.roomIcon,
@@ -574,7 +576,7 @@ Notifications.merge = async function (notifications) {
const type = typeFromLength(usernames);
const isMultiple = type === 'multiple';
const txArgs = [
`${mergeId}-${type}`,
`${mergeId}${type ? '-type' : ''}`,
...usernames.slice(0, usernames.length <= 3 ? 3 : 2),
...(isMultiple ? [usernames.length - 2] : []),
utils.decodeHTMLEntities(notifObj.topicTitle || ''),