Skip to content

Commit 9f03766

Browse files
fixup! chore(organization): sync organization data into firebase.
1 parent acc9c96 commit 9f03766

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

apps/project/firebase.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ def handle_new_organization_on_firebase(organization: Organization, organization
3333
)
3434

3535

36-
def handle_organization_update_on_firebase(organization: Organization, organization_ref: FbReference):
36+
def handle_organization_update_on_firebase(organization_ref: FbReference, organization: Organization):
3737
organization_ref.update(
3838
value=firebase_utils.serialize(
3939
firebase_models.FbOrganisation(
@@ -75,8 +75,7 @@ def push_organization_to_firebase(organization_id: int):
7575
extra=log_extra({"organization": organization.pk}),
7676
)
7777
raise InvalidOrganizationPushException
78-
valid_organization = firebase_models.FbOrganization.model_validate(obj=fb_organization)
79-
handle_organization_update_on_firebase(organization_ref, valid_organization)
78+
handle_organization_update_on_firebase(organization_ref, organization)
8079
except InvalidOrganizationPushException:
8180
organization.update_firebase_push_status(FirebasePushStatusEnum.FAILED)
8281
except Exception:

apps/project/serializers.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -355,3 +355,9 @@ def create(self, validated_data: dict[str, typing.Any]) -> Organization:
355355
organization = super().create(validated_data)
356356
transaction.on_commit(lambda: push_organization_to_firebase.delay(organization.pk))
357357
return organization
358+
359+
@typing.override
360+
def update(self, instance: Organization, validated_data: dict[typing.Any, typing.Any]):
361+
organization = super().update(instance, validated_data)
362+
transaction.on_commit(lambda: push_organization_to_firebase.delay(organization.pk))
363+
return organization

0 commit comments

Comments
 (0)