Set FROM field on emails (#55)

Fixes #54

Reviewed-on: data.coop/membersystem#55
Co-authored-by: Benjamin Bach <benjamin@overtag.dk>
Co-committed-by: Benjamin Bach <benjamin@overtag.dk>

src/accounting/models.py

@@ -80,8 +80,8 @@ class Order(CreatedModifiedAbstract):
     is_paid = models.BooleanField(default=False, verbose_name=_("is paid"))
 
     class Meta:
-        verbose_name = pgettext_lazy("accounting term", "Order")
+        verbose_name = pgettext_lazy("accounting", "Order")
-        verbose_name_plural = pgettext_lazy("accounting term", "Orders")
+        verbose_name_plural = pgettext_lazy("accounting", "Orders")
 
     def __str__(self) -> str:
         return f"Order ID {self.display_id}"

src/accounting/templates/accounting/order/detail.html

@@ -2,7 +2,7 @@
 {% load i18n %}
 
 {% block head_title %}
-    {% trans "Order" %}
+    {% trans "Order" context "accounting" %}
 {% endblock %}
 
 {% block content %}
@@ -11,14 +11,14 @@
         <h2>Order: {{ order.id }}</h2>
 
         <p>
-          {% trans "Ordered" %}: {{ order.created }}<br>
+          {% trans "Ordered" context "accounting" %}: {{ order.created }}<br>
-          {% trans "Status" %}: {{ order.is_paid|yesno:_("paid,unpaid") }}
+          {% trans "Status" context "accounting" %}: {{ order.is_paid|yesno:_("paid,unpaid") }}
         </p>
 
         <table class="table">
             <thead>
             <tr>
-                <th>{% trans "Item" %}</th>
+                <th>{% trans "Item" context "accounting" %}</th>
                 <th>{% trans "Quantity" %}</th>
                 <th>{% trans "Price" %}</th>
                 <th>{% trans "VAT" %}</th>

src/membership/emails.py

@@ -56,7 +56,7 @@ class BaseEmail(EmailMessage):
         c["request"] = self.request
         c["domain"] = site.domain
         c["site_name"] = site.name
-        c["protocol"] = "http" if self.request and not self.request.is_secure() else "https"
+        c["protocol"] = "https"  # if self.request and not self.request.is_secure() else "https"
         return c
 
     def get_body(self) -> str:
@@ -101,6 +101,7 @@ class InviteEmail(BaseEmail):
     def __init__(self, membership: Membership, request: HttpRequest, *args, **kwargs) -> None:
         self.membership = membership
         kwargs["user"] = membership.user
+        kwargs["from_email"] = "kasserer@data.coop"
         super().__init__(request, *args, **kwargs)
 
     def get_context_data(self) -> dict:
@@ -118,6 +119,7 @@ class OrderEmail(BaseEmail):
     def __init__(self, order: Order, request: HttpRequest, *args, **kwargs) -> None:
         self.order = order
         kwargs["user"] = order.member
+        kwargs["from_email"] = "kasserer@data.coop"
         super().__init__(request, *args, **kwargs)
 
     def get_context_data(self) -> dict:

src/membership/templates/membership/emails/order.txt

@@ -10,7 +10,7 @@ If you received this email and no longer want a membership, you can ignore it. B
 
 Dansk:
 
-Hej! Så kører medlemsystemet endeligt! Det er mega-fedt, fordi vi længe har haft besvær med manuelle procedurer. Nu har vi flyttet medlemsdata over på members.data.coop, og betalingen fungerer. Vi kan dermed fremover arbejde stille og roligt på at integrere systemet, så man kan styre sine services via medlemssystemet.
+Hej! Så kører medlemsystemet endeligt! Det er mega-fedt, fordi vi længe har haft besvær med manuelle procedurer. Nu har vi flyttet medlemsdata over på member.data.coop, og betalingen fungerer. Vi kan dermed fremover arbejde stille og roligt på at integrere systemet, så man kan styre sine services via medlemssystemet.
 
 Hvis du ikke længere vil være medlem, kan du ignorere mailen her; men du må meget gerne informere os via board@data.coop, så vi kan slette evt. personlige data og services, du har kørende på dit tidligere medlemskab.