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

Vylepšení hodnotítka fix #1354 fix #1237 #20

Merged
zelvuska merged 25 commits from vylepseni_odevzdavatka into master 2023-06-12 22:17:08 +02:00
Conversation 3 Commits 25 Files changed 7 +6 -1
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit c8516d6eda - Show all commits

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

@ -62,6 +62,8 @@
<h3>Neveřejná poznámka:</h3>
<p>{{ poznamka_form.poznamka }}</p>
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-02-06 22:09:49 +01:00
Outdated
Review
Copy link

LOL komentáře. Za pár let je nikdo nepochopí, ale co už :-)

LOL komentáře. Za pár let je nikdo nepochopí, ale co už :-)
zelvuska commented 2023-02-06 22:14:55 +01:00
Outdated
Review
Copy link

Na Apríla to změnit na zakomentované (a přidat popisky ❤ životy, 🔵 mana, 💪 síla, 📖 charisma)…

Na Apríla to změnit na zakomentované (a přidat popisky ❤ životy, 🔵 mana, 💪 síla, 📖 charisma)…
<script>vporadku=true;</script>
{% for h in hodnoceni %}{% if h.body < 0.0 %} <script>if(vporadku){vporadku=false;alert("Pozor! Některé hodnocení má záporné body.\n\nBuď jde o záměr, nebo o špatné zadáný počet bodů (např. součet bodů za úlohu) nebo se něco pokazilo.\n\nPokud se to děje neočekávaně a opakovaně, napiš webařům :)");}</script> {% endif %}{% endfor %}
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-02-23 12:53:53 +01:00
Outdated
Review
Copy link

Nechceme to trochu pozalamovat?

Nechceme to trochu pozalamovat?
{# Hodnocení: #}
<h3>Hodnocení:</h3>
<table>

5
odevzdavatko/views.py
View file

@ -315,7 +315,10 @@ def hodnoceniReseniView(request, pk, *args, **kwargs):
zmeny_bodu = [it for it in form.changed_data if it.startswith("body")]
if len(zmeny_bodu) == 1:
hodnoceni.__setattr__(zmeny_bodu[0], data_for_body[zmeny_bodu[0]])
hodnoceni.save()
if len(zmeny_bodu) != 1 and len(zmeny_bodu) != 4:
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-02-25 14:19:28 +01:00
Outdated
Review
Copy link

Proč že můžou být 4?

Proč že můžou být 4?
ledoian commented 2023-02-25 14:21:15 +01:00
Outdated
Review
Copy link

Už to vidím: je to 4 když se nic nezmění. Prosím, napišme to k tomu do komentáře, rozhodně mi to nepřijde intuitivní.

Už to vidím: je to 4 když se nic nezmění. Prosím, napišme to k tomu do komentáře, rozhodně mi to nepřijde intuitivní.
print(f"Hodnocení {hodnoceni} mělo mít nastavené víc různých bodů: {zmeny_bodu}. Nastavuji -0.1.")
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-02-25 14:17:54 +01:00
Outdated
Review
Copy link

Tenhle print je k něčemu dobrý? Nechal bych jen warning o řádek níž, je o dost víc konfigurovatelný…

Tenhle print je k něčemu dobrý? Nechal bych jen warning o řádek níž, je o dost víc konfigurovatelný…
logger.warning(f"Hodnocení {hodnoceni} mělo mít nastavené víc různých bodů: {zmeny_bodu}. Nastavuji -0.1.")
hodnoceni.body = -0.1
hodnoceni.save()
return redirect(success_url)