long rebate email fix

Author: mittal-ishaan

home/signals.py

@@ -105,8 +105,8 @@ def update_long_bill(sender, instance: LongRebate, **kwargs):
     logger.info("Signal called for Updating Long Rebate")
     try:
         old_instance = LongRebate.objects.get(pk=instance.pk)
-        reason = instance.reason if instance.reason is not None else "No reason"
-        logger.info(instance.approved, reason)
+        reason = instance.reason if instance.reason else "No reason"
+        logger.info(f"Approved: {instance.approved}, Reason: {reason}")
         if (
             old_instance.approved != instance.approved
             or old_instance.reason != instance.reason

home/utils/django_email_server.py

@@ -99,9 +99,9 @@ def long_rebate_mail(
     # elif(1):
     #    message = message_long_rebate.format(start_date=start_date, end_date=end_date, approved = "rejected",added="added to")
     else:
-        rejected_message = (
-            "Your rebate is not approved for the following reason: " + reason
-        )
+        rejected_message = "Your rebate is not approved."
+        if reason:
+            rejected_message = "Reason: " + reason
         message = message_long_rebate.format(
             start_date=start_date,
             end_date=end_date,