Problem/Motivation

We need a logo for project browser

Proposed resolution

Add the logo

Remaining tasks

-

CommentFileSizeAuthor
#3 3401460.patch124 bytesborisson_

Comments

borisson_ created an issue. See original summary.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented

See https://www.drupal.org/project/project_browser/issues/3342533

https://www.drupal.org/project/project_browser/issues/3277516

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
StatusFileSize
new 3401460.patch124 bytes
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
Status: Active » Needs review
bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Needs review » Reviewed & tested by the community
Parent issue: » #3277516: Search API Project - Update Logo

Checked the patch and looks good.

Also linking to the relevant issue.

drunken monkey credited amourow.

drunken monkey credited jwineichen.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Reviewed & tested by the community » Fixed

Thanks a lot for pointing me to that issue, I wouldn’t have been aware of it otherwise. (Though I was aware that logos were now supported, and did have a vague plan of adding one at some point.) Still, I hope it’s clear that I’m crediting the two people who actually worked on the logo.

Checked the patch and looks good.

Nice try. ;P

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented

Nice try. ;P

Not trying anything actually. Just triaging issues, checking file size and recommendations and if the file is in the correct location and has the correct name... not sure what more you expect for a review of such trivial thing...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.