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 +2 -2
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit f6cb669277 - Show all commits

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

@ -18,7 +18,7 @@
{% for r in object.resitele.all %}
{{ r }}
{# DjangoTemplates neumí spojovat řetězce (https://stackoverflow.com/q/4386168), tak si necháváme vyrobit subject mailu ve view. #}
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)
({% maillink r.osoba.email to=r.osoba.email subject=mailsubject %}){% if forloop.revcounter0 != 0 %}, {% endif %}
({% maillink r.osoba.email to=r.osoba.email subject=predmetmailu %}){% if forloop.revcounter0 != 0 %}, {% endif %}
{% endfor %}
</p>
{% else %}

2
odevzdavatko/views.py
View file

@ -239,7 +239,7 @@ class DetailReseniView(DetailView):
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
ctx["predmetmailu"] = "Oprava řešení M&M "+self.reseni.problem.first().hlavni_problem.nazev
return ctx
def get(self, request, *args, **kwargs):