mam/mamweb
11
0
Fork 0
Code Issues Pull requests 9 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 +1
1 changed files with 1 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 65cd15ecbb - Show all commits

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

@ -17,6 +17,7 @@
<p>Řešitelé:
{% 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 %}
{% endfor %}
</p>