Merge branch 'sales_features' into 'dev'

finance email

See merge request !274

finance/models.py

@@ -3,8 +3,6 @@ from django.conf import settings
 from django.db import models
 from yandex_money.models import Payment
 
-from courses.models import Course, Lesson
-
 
 class Bill(models.Model):
     course_token = models.UUIDField(verbose_name="Токен курса", editable=False)
@@ -62,6 +60,11 @@ class Invoice(models.Model):
     is_open = models.BooleanField(default=True, verbose_name="Открывает ли платёж курс")
     date = models.DateTimeField(auto_now_add=True)
 
+    def get_comment(self):
+        return '''Вам выставлен счёт, для оплаты перейдите по ссылке
+            %s/api/v1/finance/payment/%s/''' % (settings.DOMAIN, self.yandex_pay.id,) if\
+            self.comment is None else self.comment
+
     def __str__(self):
         return '%s:%s %s' % (self.id, self.get_status_display(), self.bill.user)
 

finance/signals.py

@@ -19,8 +19,7 @@ def invoice_signal(instance, **kwargs):
     if instance.yandex_pay and instance.method == 'Y' and instance.status == 'P':
         msg = EmailMessage(
             'Вам выставлен новый счёт',
-            '''Вам выставлен счёт, для оплаты перейдите по ссылке
-            %s/api/v1/finance/payment/%s/''' % (settings.DOMAIN, instance.yandex_pay.id,),
+            instance.get_comment(),
             'robo@skillbox.ru',
             [instance.yandex_pay.cps_email],
             [instance.bill.opener.email],