Merge pull request #382 from brunoamaral/base-commands

remove redundant setting of variables
brunoamaral · Mar 29, 2024 · 294817e · 294817e
2 parents eec0ffe + 769d913
commit 294817e
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/django/subscriptions/management/commands/send_admin_summary.py b/django/subscriptions/management/commands/send_admin_summary.py
@@ -44,9 +44,10 @@ def handle(self, *args, **options):
                 trial.sent_to_admin = True
             Trials.objects.bulk_update(trials, ['sent_to_admin'])
 
-    def send_simple_message(self, sender='Gregory MS <gregory@mg.' + Site.objects.get_current().domain + '>', to=None,bcc=None,subject='no subject', text=None,html=None, email_mailgun_api_url=settings.EMAIL_MAILGUN_API_URL, email_mailgun_api=settings.EMAIL_MAILGUN_API):
-        email_mailgun_api_url = settings.EMAIL_MAILGUN_API_URL
-        email_mailgun_api = settings.EMAIL_MAILGUN_API
+    def send_simple_message(self, to, bcc=None, subject=None, text=None, html=None, 
+                            sender=f'Gregory MS <gregory@mg.{Site.objects.get_current().domain}>',
+                            email_mailgun_api_url=settings.EMAIL_MAILGUN_API_URL, 
+                            email_mailgun_api=settings.EMAIL_MAILGUN_API):
         print(f"data=sender: {sender}, to: {to}, bcc: {bcc}")
         status = requests.post(
             email_mailgun_api_url,