Nešlo by to udělat, aby to ignorovalo case?
A to by se mělo udělat i u odpovědi, co?
Spíš se mi z uživatelského pohledu nelíbí get_object_or_404, protože to ukáže generickou věc a ne specifické vysvětlení…
Nechceme ještě něco jako vymaž všechny šifry z daného seznamu? (Jakože aktuálně musí člověk promazat šifry z minulého sousu pěkně jednu po druhé…)
Můžeš smazat seznam,…
Jo, a mám tam zamergeovaný !93, pokud to chcete rozdělit, tak to klidně rozdělím (ty commity by měly být příčetné, jen dělám koniny)
Teda, mám otestováno jen odevzdávání správných a špatných odpovědí, zatím nevím, co všechno dalšího to má umět, byť nečekám, že to někde selže…
To je podle mě asi jedno, spíš mi chybí, exporty čeho to jsou, případně v jakém formátu…
Tyhle ify vypadají dost zběsile. Chápu správně, že se prakticky jen mění, jaký QuerySet předhodíme dataOsobCsvResponse a pak možná filename?
Nevidím evidentní chyby, ale mám z toho kódu dost pocit, že sahat do toho bude strašně nepohodlné a náchylné na chyby, hlavně kvůli kopírovanému kódu, předávání magických číselných konstant a nezamýšlení se nad chybovými stavy.
Nechceš použít fields.join(",")? Tohle mě neuráží, ale .join zní jakože to ušetří 5 řádků a dvě logické chyby…
Vlastně mi přijde, že je tam klidně NějakouDobu™ můžeme nechat – pokud někomu fungovaly / vyhovovaly staré exporty lépe (nevím přesně proč a jak), tak ať je klidně ještě…
Noooo, asi to lepší nebude… Ideálně někam poznamenat moje výtky k dokumentaci, ale jinak asi mergenout…
Přijde mi dobré mít v tomhle místě při čtení kódu mít kontext, že template tu Znalost rozepisuje nezávisle, a tím pádem ve formuláři se má vložit neviditelně.
Ale je to…