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

Výroční sraz M&M #32

Merged
zelvuska merged 12 commits from vyroci into master 2023-06-15 16:57:27 +02:00
Conversation 1 Commits 12 Files changed 13 +10 -10
2 changed files with 10 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit e6d4bfe0f5 - Show all commits

4
vyroci/templates/vyroci/vyroci.html
View file

@ -13,7 +13,7 @@
<ul> <ul>
<li> Kdy: pátek 22. – neděle 24. 9. 2023 </li> <li> Kdy: pátek 22. – neděle 24. 9. 2023 </li>
<li> Kde: <a href="https://borovice.cz/inzerat/2587-zakladna-pratel-prirody-upirci-2">Klubovna Upírků v Libčicích n.V.</a></li> <li> Kde: <a href="https://borovice.cz/inzerat/2587-zakladna-pratel-prirody-upirci-2">Klubovna Upírků v Libčicích n.V.</a></li>
<li> Kdo jede: Káťa, (R)adim{% for ucastnik in ucastnici %}, {{ ucastnik.prezdivka }}{% endfor %}</li> <li> Kdo jede: {% for ucastnik in ucastnici %}{% if not forloop.first %}, {% endif %}{{ ucastnik.prezdivka }}{% endfor %}</li>
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-06-13 19:02:42 +02:00
Outdated
Review
Copy link

Nevím, jaká je dohoda, ale možná je spíš žádoucí, aby byli všichni účastníci v databázi, protože jinak se na Káťu a (R)adima bude snadno zapomínat a nebude to vidět.

Klidně bych je rovnou přidal v rámci migrace (a do kontextu pak klidně narval seznam_ucastniku = ', '.join(u.prezdivka for u in ucastnici)

Nevím, jaká je dohoda, ale možná je spíš žádoucí, aby byli všichni účastníci v databázi, protože jinak se na Káťu a (R)adima bude snadno zapomínat a nebude to vidět. Klidně bych je rovnou přidal v rámci migrace (a do kontextu pak klidně narval `seznam_ucastniku = ', '.join(u.prezdivka for u in ucastnici)`
zelvuska commented 2023-06-13 22:39:27 +02:00
Outdated
Review
Copy link

Přijde mi, že jednodušší bude je tam naházet ručně po nasazení…

Přijde mi, že jednodušší bude je tam naházet ručně po nasazení…
ledoian commented 2023-06-13 22:41:31 +02:00
Outdated
Review
Copy link

Klidně, když se na to nezapomene (nebo když těm dvěma někdo řekne, že se mají prostě normálně zaregistrovat), spíš mi jde o to, aby tady nebyli nahardcodění, protože „určitě pojedou“.

Klidně, když se na to nezapomene (nebo když těm dvěma někdo řekne, že se mají prostě normálně zaregistrovat), spíš mi jde o to, aby tady nebyli nahardcodění, protože „určitě pojedou“.
</ul> </ul>
<h2>Přihlašovací formulář</h2> <h2>Přihlašovací formulář</h2>
@ -22,6 +22,7 @@
<form action="{% url 'vyrocni_sraz' %}" method="post"> <form action="{% url 'vyrocni_sraz' %}" method="post">
<table class="form"> <table class="form">
{{form.non_field_errors}}
{% for field in form %} {% for field in form %}
<tr> <tr>
<td> <td>
@ -48,7 +49,6 @@
</table> </table>
{% csrf_token %} {% csrf_token %}
zelvuska marked this conversation as resolved
ledoian commented 2023-06-13 19:03:23 +02:00
Review
Copy link

Non-field errors bych čekal spíš nahoře než dole…

Non-field errors bych čekal spíš nahoře než dole…
{{form.non_field_errors}}
<input type="submit" value="Přihlásit se!"> <input type="submit" value="Přihlásit se!">
</form> </form>

16
vyroci/templates/vyroci/vyroci_list.html
View file

@ -15,15 +15,15 @@
<th>Něco dodat?</th> <th>Něco dodat?</th>
</tr> </tr>
{% for obj in ucastnikvyroci_list %} {% for u in object_list %}
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-06-13 19:10:03 +02:00
Outdated
Review
Copy link

Tohle funguje? Měl jsem za to, že ListView vyrábí v kontextu object_list… (A možná bych to malinko preferoval, kvůli konzistenci; byť by asi rozumné naopak bylo plošně všechny object_list přepsat na to, seznamy čeho to reálně jsou. Asi spíš výhledově, do Kanboardu…)

Jo, a když už jsme u toho, tak bych neobjektifikoval účastníky a tu iterační proměnou pojmenoval třeba u. Ale taky je to spíš taková poznámka, která by se spíš týkala nějaké dlouhodobější stránky, než téhle…

Tohle funguje? Měl jsem za to, že `ListView` vyrábí v kontextu `object_list`… (A možná bych to malinko preferoval, kvůli konzistenci; byť by asi rozumné naopak bylo plošně všechny `object_list` přepsat na to, seznamy čeho to reálně jsou. Asi spíš výhledově, do Kanboardu…) Jo, a když už jsme u toho, tak bych neobjektifikoval účastníky a tu iterační proměnou pojmenoval třeba `u`. Ale taky je to spíš taková poznámka, která by se spíš týkala nějaké dlouhodobější stránky, než téhle…
<tr> <tr>
<td>{{ obj.jmeno }}</td> <td>{{ u.jmeno }}</td>
<td>{{ obj.prezdivka }}</td> <td>{{ u.prezdivka }}</td>
<td>{{ obj.email }}</td> <td>{{ u.email }}</td>
<td style="word-break: break-all">{{ obj.kdy }}</td> <td style="word-break: break-all">{{ u.kdy }}</td>
<td style="word-break: break-all">{{ obj.cojemam }}</td> <td style="word-break: break-all">{{ u.cojemam }}</td>
<td style="word-break: break-all">{{ obj.cislo }}</td> <td style="word-break: break-all">{{ u.cislo }}</td>
<td style="word-break: break-all">{{ obj.dodat }}</td> <td style="word-break: break-all">{{ u.dodat }}</td>
</tr> </tr>
{% endfor %} {% endfor %}
</table> </table>