Skip to content

fix(comments): Add an action to comment notification that dismisses it - #60150

Merged
kesselb merged 1 commit into
masterfrom
bugfix/noid/add-action-to-notification-to-mark-read
May 5, 2026
Merged

fix(comments): Add an action to comment notification that dismisses it#60150
kesselb merged 1 commit into
masterfrom
bugfix/noid/add-action-to-notification-to-mark-read

Conversation

@nickvergessen

@nickvergessennickvergessen commented May 4, 2026

Copy link
Copy Markdown
Member

Summary

BeforeAfter
grafikgrafik

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@nickvergessennickvergessen added this to the Nextcloud 34 milestone May 4, 2026
@nickvergessennickvergessen self-assigned this May 4, 2026
@nickvergessen
nickvergessen requested a review from a team as a code ownerMay 4, 2026 20:28
@nickvergessen
nickvergessen requested review from leftybournes, nfebe and salmart-dev and removed request for a teamMay 4, 2026 20:28
@nickvergessennickvergessen added the 3. to review Waiting for reviews label May 4, 2026
@nickvergessennickvergessen added papercut Annoying recurring UX issue with possibly simple fix. feature: comments labels May 4, 2026
@nickvergessen

nickvergessen commented May 4, 2026

Copy link
Copy Markdown
MemberAuthor

/backport to stable33

@nickvergessen

nickvergessen commented May 4, 2026

Copy link
Copy Markdown
MemberAuthor

/backport to stable32

@miaulalala
miaulalalaforce-pushed the bugfix/noid/add-action-to-notification-to-mark-read branch from e846097 to 81a30c7CompareMay 5, 2026 10:18
@nickvergessen
fix(comments): Add an action to comment notification that dismisses it
2e41ee8
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the bugfix/noid/add-action-to-notification-to-mark-read branch from 81a30c7 to 2e41ee8CompareMay 5, 2026 12:10
@kesselb
kesselb merged commit e622901 into masterMay 5, 2026
196 of 200 checks passed
@kesselb
kesselb deleted the bugfix/noid/add-action-to-notification-to-mark-read branch May 5, 2026 21:47
This was referenced May 5, 2026
Merged
Merged
This was referenced May 7, 2026
Merged
Merged
@nextcloud-botnextcloud-bot mentioned this pull request May 15, 2026
Merged
@miaulalala

miaulalala commented May 19, 2026

Copy link
Copy Markdown
Contributor

/backport to stable31

@backportbotbackportbotBot mentioned this pull request May 19, 2026
Merged
@miaulalalamiaulalala mentioned this pull request May 19, 2026
Closed
7 tasks
@nextcloud-botnextcloud-bot mentioned this pull request May 20, 2026
Merged
@nextcloud-botnextcloud-bot mentioned this pull request May 27, 2026
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kesselbkesselbkesselb approved these changes
@miaulalalamiaulalalamiaulalala approved these changes
@janepiejanepieAwaiting requested review from janepie
@salmart-devsalmart-devAwaiting requested review from salmart-devsalmart-dev is a code owner automatically assigned from nextcloud/server-backend
@nfebenfebeAwaiting requested review from nfebenfebe is a code owner automatically assigned from nextcloud/server-backend
@leftybournesleftybournesAwaiting requested review from leftybournesleftybournes is a code owner automatically assigned from nextcloud/server-backend
@sorbaughsorbaughAwaiting requested review from sorbaughsorbaugh is a code owner automatically assigned from nextcloud/server-backend

Assignees

@nickvergessennickvergessen

Labels

3. to reviewWaiting for reviewsbugfeature: commentspapercutAnnoying recurring UX issue with possibly simple fix.

Projects

None yet

Milestone

Nextcloud 34

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen@miaulalala@kesselb