fix: remove logo and favicon default#28
Conversation
There was a problem hiding this comment.
Code Review
This pull request refactors the OrganizationThemeSerializer by replacing the SerializerMethodField fields (url_logo and url_favicon) and their respective getter methods with a custom to_representation method. This method dynamically injects the presigned S3 URLs for the logo and favicon if they are configured, and removes the fallback logic that previously returned default local static asset URLs. I have no feedback to provide as there are no review comments.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
What?
Remove logo and favicon default
Why?
From the request
How?
Testing?
Anything Else?