Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Restore user.svg icon as it's still used in notifications #994

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Restore user.svg icon as it's still used in notifications #994

merged 1 commit into from
Nov 10, 2023

Conversation

nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Nov 10, 2023 •
edited
Loading

Ref https://github.com/nextcloud/firstrunwizard/pull/960/files#r1389519541

Still used in

firstrunwizard/lib/Notification/Notifier.php

Line 106 in 559151d

->setIcon($this->url->getAbsoluteURL($this->url->imagePath('firstrunwizard', 'user.svg')));

@nickvergessen nickvergessen self-assigned this Nov 10, 2023
@nickvergessen nickvergessen merged commit 9179442 into master Nov 10, 2023
17 checks passed
@nickvergessen nickvergessen deleted the bugfix/960/restore-user-icon branch November 10, 2023 15:14
@blizzz blizzz mentioned this pull request Nov 13, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juliusknorr juliusknorr juliusknorr approved these changes

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

Assignees

@nickvergessen nickvergessen

Labels
3. to review bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nickvergessen @AndyScherzinger @juliusknorr