Skip to content

[stable31] fix(notification): Fix color of notification icon in browser notifications#2874

Merged
nickvergessen merged 2 commits intostable31from
backport/2862/stable31
Mar 24, 2026
Merged

[stable31] fix(notification): Fix color of notification icon in browser notifications#2874
nickvergessen merged 2 commits intostable31from
backport/2862/stable31

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot bot commented Mar 24, 2026

Backport of #2862

Warning, This backport had conflicts that were resolved with the ours merge strategy and is likely incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Comment thread src/services/webNotificationsService.js Outdated
…tions

Signed-off-by: Joas Schilling <coding@schilljs.com>
@Antreesy Antreesy force-pushed the backport/2862/stable31 branch from 1ff1d42 to 08ff00e Compare March 24, 2026 10:23
@nickvergessen
Copy link
Copy Markdown
Member

/compile /

@nextcloud nextcloud deleted a comment from artonge Mar 24, 2026
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nickvergessen nickvergessen marked this pull request as ready for review March 24, 2026 10:25
Copy link
Copy Markdown
Collaborator

@Antreesy Antreesy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image

@nickvergessen nickvergessen merged commit c39cfb0 into stable31 Mar 24, 2026
43 of 44 checks passed
@nickvergessen nickvergessen deleted the backport/2862/stable31 branch March 24, 2026 10:29
@artonge
Copy link
Copy Markdown
Contributor

artonge commented Apr 1, 2026

@nickvergessen or @Antreesy any chance we can have a new release for 31 containing this fix?

@nickvergessen
Copy link
Copy Markdown
Member

its a shipped app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants