Bug #22966
closedSite ID pictures not appearing in Commons site directory
0%
Description
Hi All,
On a call earlier we found that some site ID images are not appearing in the site directory - instead replaced by the words "Site icon for [site title]."
I believe images not appearing should be pulling from the Site Identity "icon", which users input in the customize area of the site. I am wondering if maybe the sites where this is not working are possibly private sites, and something is wrong with the image pulls from the server?
Let me know what you think, thanks!
Files
Updated by Raymond Hoh 24 days ago
- Category name set to S3 Uploads
- Status changed from New to Resolved
- Target version set to 2.5.11
I am wondering if maybe the sites where this is not working are possibly private sites, and something is wrong with the image pulls from the server?
This is exactly what is happening. I've just added a fix so the S3 signed parameters are added to the site icon image URL to make private site icons viewable on the Site Directory page: https://github.com/cuny-academic-commons/cac/commit/9e2c48270e84c1d91c9cddf559788bbbd37906a5. This is deployed to production.
Updated by Laurie Hurson 15 days ago
- File Screenshot 2025-07-02 at 12.32.32 PM.png Screenshot 2025-07-02 at 12.32.32 PM.png added
- File Screenshot 2025-07-02 at 12.33.29 PM.png Screenshot 2025-07-02 at 12.33.29 PM.png added
I am still seeing this issue, screenshots attached.
Updated by Raymond Hoh 15 days ago
Thanks for the report, Laurie.
I tried to limit the calls to look up the site's privacy settings for performance reasons, but that doesn't work all the time, so I'm fetching the site's privacy settings at all times now in https://github.com/cuny-academic-commons/cac/commit/e1aa8b709f144d7599f737f67a0ec50e0f4fb76d. This is deployed to production.
Boone, to make the private site icon call more optimal, we might need to start mirroring the 'blog_public'
option to BP blogmeta, then we will not need to do a switch_to_blog()
call for every site in the Sites Directory. What do you think?
Updated by Boone Gorges 9 days ago
Boone, to make the private site icon call more optimal, we might need to start mirroring the 'blog_public' option to BP blogmeta, then we will not need to do a switch_to_blog() call for every site in the Sites Directory. What do you think?
I think it's a fine idea - kinda surprised we don't already do this.