Skip to content

fix: Organization logo is now cleared when update sends explicit null - #123

Merged
m-t-a97 merged 1 commit into
mainfrom
fix/organization-logo-removal
Oct 10, 2026
Merged

m-t-a97 merged 1 commit into
mainfrom
fix/organization-logo-removal

Conversation

@m-t-a97

@m-t-a97 m-t-a97 commented Oct 10, 2026 •

Copy link
Copy Markdown
Member

The update-organization endpoint could not tell an explicit logo: null apart from an omitted logo field, so clearing a logo left the old value in place. UpdateOrganizationRequest now records whether the logo field was present in the incoming JSON and the service clears the logo to nil for an explicit null or empty string.

@m-t-a97 m-t-a97 self-assigned this Oct 10, 2026
@m-t-a97
m-t-a97 force-pushed the fix/organization-logo-removal branch 2 times, most recently from 137b919 to efb1927 Compare October 10, 2026 00:16
The update-organization endpoint could not tell an explicit logo: null apart from an omitted logo field, so clearing a logo left the old value in place. UpdateOrganizationRequest now records whether the logo field was present in the incoming JSON and the service clears the logo to nil for an explicit null or empty string.
@m-t-a97
m-t-a97 force-pushed the fix/organization-logo-removal branch from efb1927 to 0a10bc3 Compare October 10, 2026 00:17
@m-t-a97 m-t-a97 changed the title fix: Add support for clearing organization logo when update sends exp… fix: Organization logo is now cleared when update sends explicit null Oct 10, 2026
@m-t-a97
m-t-a97 merged commit 14dfcd4 into main Oct 10, 2026
6 checks passed
@m-t-a97
m-t-a97 deleted the fix/organization-logo-removal branch October 10, 2026 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant