mam/mamweb
11
0
Fork 0
Code Issues Pull requests 8 Projects Wiki Activity Actions

Tag pro odkazy na poslání mailu #22

Merged
ledoian merged 7 commits from maillink into master 2023-02-06 22:02:00 +01:00
Conversation 11 Commits 7 Files changed 5 +8 -1
2 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit efe1b4bb5a - Show all commits

6
odevzdavatko/templates/odevzdavatko/detail.html
View file

@ -1,6 +1,7 @@
{% extends "base.html" %}
{% load static %}
{% load deadliny %}
{% load mail %}
{% block content %}
@ -14,7 +15,10 @@
{% if edit %}
<p>Řešitelé:
{% for r in object.resitele.all %}{{ r }} (<a href="mailto:{{ r.osoba.email }}?subject={{ "Oprava řešení M&M " | urlencode }}{{ object.problem.all.0.hlavni_problem | urlencode }}">{{ r.osoba.email }}</a>){% if forloop.revcounter0 != 0 %}, {% endif %}{% endfor %}
{% for r in object.resitele.all %}
{{ r }}
({% maillink r.osoba.email to=r.osoba.email subject=mailsubject %}){% if forloop.revcounter0 != 0 %}, {% endif %}
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-01-08 18:10:26 +01:00
Outdated
Review
Copy link

Přemýšlím, jestli to nechce komentář, proč máme mailsubject generovaný ve view, když je to prakticky čistě prezentační záležitost… Názory?

Přemýšlím, jestli to nechce komentář, proč máme `mailsubject` generovaný ve view, když je to prakticky čistě prezentační záležitost… Názory?
zelvuska commented 2023-02-06 19:35:35 +01:00
Outdated
Review
Copy link

A možná v template komentář, kde se vzal mailsubject / co to je?

A možná v template komentář, kde se vzal mailsubject / co to je?
ledoian commented 2023-02-06 19:51:25 +01:00
Outdated
Review
Copy link

Template IMHO spíš komentář nepotřebuje. Je to evidentně subject mailu, a věci se typicky berou v kontextu (a v development toolbaru by měl být normálně vidět). Ta divná věc je to, že ho bastlíme ve view…

Template IMHO spíš komentář nepotřebuje. Je to evidentně subject mailu, a věci se typicky berou v kontextu (a v development toolbaru by měl být normálně vidět). Ta divná věc je to, že ho bastlíme ve view…
ledoian commented 2023-02-06 19:52:15 +01:00
Outdated
Review
Copy link

Ale možná by se mohl jmenovat česky, není důvod to tak neudělat (kromě toho, že jsem to nejspíš kódil v tramvaji a tam kódím většinou anglicky :-D)

Ale možná by se mohl jmenovat česky, není důvod to tak neudělat (kromě toho, že jsem to nejspíš kódil v tramvaji a tam kódím většinou anglicky :-D)
{% endfor %}
</p>
{% else %}
<p>Řešitelé: {{ object.resitele.all | join:", " }}</p>

3
odevzdavatko/views.py
View file

@ -237,6 +237,9 @@ class DetailReseniView(DetailView):
ctx = super().get_context_data(**kw)
detaily_hodnoceni = self.aktualni_hodnoceni()
ctx["hodnoceni"] = detaily_hodnoceni
# Subject případného mailu (template neumí použitelně spojovat řetězce: https://stackoverflow.com/q/4386168)
ctx["mailsubject"] = "Oprava řešení M&M "+self.reseni.problem.first().hlavni_problem.nazev
return ctx
def get(self, request, *args, **kwargs):