fix(comments): dismiss mention notifications and clear unread badge when viewed in activity sidebar by miaulalala · Pull Request #60617 · nextcloud/server

miaulalala · 2026-05-20T15:55:04Z

Summary

When comments are loaded via the Activity sidebar integration, call markCommentsAsRead() (DAV PROPPATCH) so the file-row unread comment bubble clears after viewing
Update the frontend file node immediately via node.update({ 'comments-unread': 0 }) so the bubble disappears without waiting for a PROPFIND refresh
Add a new DELETE /apps/comments/notifications/{id} endpoint that marks a comments/comment/mention notification as processed without redirecting
Call that endpoint for each comment that mentions the current user when the Activity sidebar loads, so the notification bell clears without the user having to navigate via the notification link

Without this, opening the Activity sidebar for a file never triggered a DAV read-marker update, so the unread comment bubble persisted in the Files list even after viewing all comments. The notification bell also kept showing comment-mention notifications after the user had already read them in the sidebar.

Fixes: nextcloud/activity#2531

Test plan

With the Activity app enabled, open a file's Activity sidebar that has unread comments — the chat bubble on the file row should disappear after the sidebar loads
If you are @mentioned in a comment and open the Activity sidebar for that file, the notification bell entry for the mention should be cleared
Confirm the new DELETE /apps/comments/notifications/{id} endpoint returns 200 for a valid comment id, 403 if not logged in, 404 for unknown ids
Run existing PHP unit tests: NOCOVERAGE=1 ./autotest.sh sqlite apps/comments/tests/Unit/Controller/NotificationsTest.php
Cypress tests: cypress/e2e/comments/comments-unread.cy.ts

🤖 Generated with Claude Code

… in activity sidebar Add a `DELETE /notifications/dismiss/{id}` endpoint to the comments NotificationsController that marks the `comments/comment/mention` notification as processed without redirecting. When comments are loaded in the activity sidebar, the frontend calls this endpoint for any comment that mentions the current user, so the notification is cleared from the bell without requiring the user to navigate via the notification link. Fixes: nextcloud/activity#2531 AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Anna Larch <anna@nextcloud.com>

miaulalala · 2026-05-20T16:16:23Z

/backport to stable34

miaulalala · 2026-05-20T16:16:29Z

/backport to stable33

…ivity sidebar Backport of #60617 to stable32. When the Activity app integration is active, comments are loaded in the Activity sidebar instead of the Comments tab. The read marker and mention notification dismissal were never triggered in this path, leaving the unread bubble in the file list and keeping mention notifications active. Also adds a new DELETE /notifications/{id} endpoint to dismiss individual mention notifications by comment ID. AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Anna Larch <anna@nextcloud.com>

…hen viewed in activity sidebar Backport of #60617 for stable31. When comments are loaded via the Activity sidebar integration, call markCommentsAsRead() so the file-row unread comment bubble clears after viewing. Also add a DELETE /notifications/{id} endpoint and call it for each comment that mentions the current user so the notification bell clears without navigating via the notification link. Fixes: nextcloud/activity#2531 AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Anna Larch <anna@nextcloud.com>

…ivity sidebar Backport of #60617 to stable32. When the Activity app integration is active, comments are loaded in the Activity sidebar instead of the Comments tab. The read marker and mention notification dismissal were never triggered in this path, leaving the unread bubble in the file list and keeping mention notifications active. Also adds a new DELETE /notifications/{id} endpoint to dismiss individual mention notifications by comment ID. AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Anna Larch <anna@nextcloud.com>

…hen viewed in activity sidebar Backport of #60617 for stable31. When comments are loaded via the Activity sidebar integration, call markCommentsAsRead() so the file-row unread comment bubble clears after viewing. Also add a DELETE /notifications/{id} endpoint and call it for each comment that mentions the current user so the notification bell clears without navigating via the notification link. Fixes: nextcloud/activity#2531 AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Anna Larch <anna@nextcloud.com>

joshtrichards

Production code changes themselves seem fine, other than the share bit already commented on.

The only other note I have is that the Activity-tab code path that detects a current-user mention and sends the dismissal DELETE has no Cypress coverage.

joshtrichards · 2026-07-30T14:26:05Z

-				// Similarly the token is always set by the backend when the
-				// share is created.
-				this.share._share.token = share.token


Not sure what this is for - is this change unrelated to the PR?

Also I have a suspicion this may be contributing the significant generated dist/ updates in this PR.

joshtrichards · 2026-07-30T14:31:33Z

+				const isMentioned = comment.props?.id && mentions.some((m) => m.mentionType === 'user' && m.mentionId === currentUser.uid)
+				if (isMentioned) {
+					axios.delete(generateUrl('/apps/comments/notifications/{id}', { id: comment.props.id }))
+						.catch(() => {})


Rather than 100% silently discarding every DAV and notification failure maybe log a debug-level message with the file/comment ID and error so there's still some observability.

joshtrichards · 2026-07-30T14:36:10Z

+			$this->markProcessed($comment, $currentUser);
+			return new DataResponse([]);
+		} catch (\Exception $e) {
+			return new DataResponse([], Http::STATUS_NOT_FOUND);


Not necessarily a blocker for this narrow endpoint, but narrowing the caught exception to comment lookup related failures would avoid masking unrelated errors as missing comments and improve diagnosability.

joshtrichards · 2026-07-30T15:04:56Z

+		cy.visit('/apps/files')
+
+		// Verify badge is present first
+		getInlineActionEntryForFile('commented-file.txt', 'comments-unread')


Is this guaranteed to exercise the Activity path (the comments-activity-tab.ts implementation updated in this PR) rather than the old comments sidebar?

server/apps/comments/src/utils/activity.ts

Lines 8 to 13 in f5b5225

/**

* Check if the comments app is using the Activity app integration for the sidebar.

*/

export function isUsingActivityIntegration() {

return loadState('comments', 'activityEnabled', false) && window.OCA?.Activity?.registerSidebarAction !== undefined

}

joshtrichards · 2026-07-30T15:07:47Z

+			.should('not.exist')
+	})
+
+	it('badge stays absent after closing and re-opening the sidebar', () => {


Says "after closing and re-opening the sidebar" but only closes it, no?

miaulalala requested review from a team as code owners May 20, 2026 15:55

miaulalala requested review from ArtificialOwl, janepie, leftybournes, nfebe, sorbaugh and susnux and removed request for a team May 20, 2026 15:55

miaulalala self-assigned this May 20, 2026

miaulalala added bug 3. to review Waiting for reviews feature: comments feature: activity and notification 31-feedback 32-feedback 33-feedback AI assisted labels May 20, 2026

miaulalala added this to the Nextcloud 35 milestone May 20, 2026

miaulalala force-pushed the fix/2531/comments-not-marked-read branch from 4142aee to 964ba1c Compare May 20, 2026 15:59

backportbot Bot added the backport-request label May 20, 2026

This was referenced May 20, 2026

[stable31] fix(comments): mark comments as read and dismiss notifications in Activity sidebar #60621

Merged

[stable32] fix(comments): mark comments as read and dismiss notifications in Activity sidebar #60622

Open

joshtrichards reviewed Jul 30, 2026

View reviewed changes

Navigation Menu

Uh oh!

Uh oh!

fix(comments): dismiss mention notifications and clear unread badge when viewed in activity sidebar - #60617

fix(comments): dismiss mention notifications and clear unread badge when viewed in activity sidebar#60617
miaulalala wants to merge 1 commit into
masterfrom
fix/2531/comments-not-marked-read

miaulalala commented May 20, 2026

Uh oh!

miaulalala commented May 20, 2026

Uh oh!

miaulalala commented May 20, 2026

Uh oh!

joshtrichards left a comment

Uh oh!

joshtrichards Jul 30, 2026

Uh oh!

joshtrichards Jul 30, 2026

Uh oh!

joshtrichards Jul 30, 2026

Uh oh!

joshtrichards Jul 30, 2026

Uh oh!

joshtrichards Jul 30, 2026

Uh oh!

joshtrichards Jul 30, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	/**
	* Check if the comments app is using the Activity app integration for the sidebar.
	*/
	export function isUsingActivityIntegration() {
	return loadState('comments', 'activityEnabled', false) && window.OCA?.Activity?.registerSidebarAction !== undefined
	}

Uh oh!

Uh oh!

Conversation

miaulalala commented May 20, 2026

Summary

Test plan

Uh oh!

miaulalala commented May 20, 2026

Uh oh!

miaulalala commented May 20, 2026

Uh oh!

joshtrichards left a comment

Choose a reason for hiding this comment

Uh oh!

joshtrichards Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

joshtrichards Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

joshtrichards Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

joshtrichards Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

joshtrichards Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

joshtrichards Jul 30, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants