From 8f59f81796654d07426f8f52329e9df3587316b4 Mon Sep 17 00:00:00 2001 From: rizwanisready Date: Mon, 12 Aug 2024 20:47:36 +0530 Subject: [PATCH] my subscriptions page 11 --- goodtimes/webhook/payment_processing_service.py | 1 - manage_subscriptions/forms.py | 13 +++++++++++++ manage_subscriptions/models.py | 6 ++++++ 3 files changed, 19 insertions(+), 1 deletion(-) diff --git a/goodtimes/webhook/payment_processing_service.py b/goodtimes/webhook/payment_processing_service.py index 4d2f7b1..5b075fd 100644 --- a/goodtimes/webhook/payment_processing_service.py +++ b/goodtimes/webhook/payment_processing_service.py @@ -105,7 +105,6 @@ class PaymentProcessingService: except Exception as e: logger.error(f"Unexpected error: {str(e)}") - self.handle_failure(txn) raise def handle_success(self, transaction): diff --git a/manage_subscriptions/forms.py b/manage_subscriptions/forms.py index 12294f5..d34f9cf 100644 --- a/manage_subscriptions/forms.py +++ b/manage_subscriptions/forms.py @@ -46,6 +46,19 @@ class SubscriptionForm(forms.ModelForm): id__in=[event_user.id, event_manager.id] ) + def clean(self): + cleaned_data = super().clean() + + stripe_product = cleaned_data.get("stripe_product") + + if not stripe_product: + self.add_error( + "stripe_product", + "Please select a Stripe product to create a subscription.", + ) + + return cleaned_data + class PrincipalSubscriptionForm(forms.ModelForm): class Meta: diff --git a/manage_subscriptions/models.py b/manage_subscriptions/models.py index 0c3ec19..21f3adf 100644 --- a/manage_subscriptions/models.py +++ b/manage_subscriptions/models.py @@ -77,6 +77,12 @@ class Subscription(BaseModel): {"high_amount": "High amount must be greater than amount."} ) + # Ensure stripe_product is compulsory present + # if not self.stripe_product: + # raise ValidationError( + # {"stripe_product": "Please select stripe product to create subscription."} + # ) + def save(self, *args, **kwargs): self.clean() # Call clean before saving to ensure validation super().save(*args, **kwargs)