From 8b9fe3c72bfee9408750d71c48a41bec342c5816 Mon Sep 17 00:00:00 2001 From: wangruidong <940853815@qq.com> Date: Thu, 10 Apr 2025 16:03:32 +0800 Subject: [PATCH] perf: add default logo handling in IntegrationApplicationSerializer --- apps/accounts/serializers/account/service.py | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/apps/accounts/serializers/account/service.py b/apps/accounts/serializers/account/service.py index eb0091b60..d94d2417e 100644 --- a/apps/accounts/serializers/account/service.py +++ b/apps/accounts/serializers/account/service.py @@ -1,3 +1,4 @@ +from django.templatetags.static import static from django.utils.translation import gettext_lazy as _ from rest_framework import serializers @@ -27,13 +28,14 @@ class IntegrationApplicationSerializer(BulkOrgResourceModelSerializer): 'name': {'label': _('Name')}, 'accounts_amount': {'label': _('Accounts amount')}, 'is_active': {'default': True}, + 'logo': {'required': False}, } - def __init__(self, *args, **kwargs): - super().__init__(*args, **kwargs) - request_method = self.context.get('request').method - if request_method == 'PUT': - self.fields['logo'].required = False + def to_representation(self, instance): + data = super().to_representation(instance) + if not data.get('logo'): + data['logo'] = static('img/logo.png') + return data class IntegrationAccountSecretSerializer(serializers.Serializer):