Předělání sousových views #57
Merged
zelvuska
merged 11 commits from predelani_sousovych_view
into master
1 week ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'predelani_sousovych_view'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Hrál jsem si se sousovými views a vznikly z toho nějaké myšlenky, které bych tu rozebíral samostatně:
KonkretniSoustredeniMixin
, který všem templatům konkrétního sousu dodává atributself.soustredeni
(aby se nemusel vždy tahat zeself.kwargs
)NeprazdnyListView
, kterýListView
přidává handlování prázdného seznamu (inteligentnější než vyhodit 404)ListView
je bohužel moc zadrátovaný, takže v tříděNeprazdnyListView
je nějaký copy-paste kód (potenciálně lze nahradit něčím jednodušším, ale méně obecným)TeXResponseMixin
, který generalizuje generování PDF z TeXuPrefix
WIP:
tu je proto, že tento pull request koliduje na #56 (speciálně to mám aktuálně jako pull request dosplit_sous
, aby byly dobře vidět změny).Možná někam napsat, že
object_list
je list objektů třídySoustredeni_Ucastnici
, protože na sobě mají ještě navíc poznámku (k danému účastníkovi na daný sous)?5f7ec853
75344c2c
WIP: Předělání sousových viewsto Předělání sousových views 4 weeks agoresponse = HttpResponse(content_type='text/csv')
response['Content-Disposition'] = 'attachment; filename="ucastnici.csv"'
writer = csv.writer(response)
Nemám to teď nacachované, ale pročže nepoužijeme generický export řešitelů, který máme už u výsledkovek?
Protože to takhle už bylo (tj. není to předmětem tohodle pullrequestu). Teď bych to nechal a můžem upravit časem.
s titlem `self.if_prazdny_title` a textem `self.if_prazdny_text`
a způsob renderování (např. CSV) lze změnit přepsáním metody render.
"""
allow_empty = False
Tohle je naše věc, nebo něco co využívá
ListView
interně?nebo jiný seznam absolutních cest).
"""
dalsi_potrebne_soubory = []
tex = "pdflatex"
Navrhuju spíš pojmenovat
tex_command
nebo i českytex_prikaz
.Asi OK, i guess. Principiálně to věci podle mě nezhoršuje, za sebe tam zas tak moc zesložitění nevidím (ale nemám optiku nových webařů…). Neříkám, že vidím úplně moc užitečnost (která další PDF kromě obálek a stvrzenek web vůbec výhledově bude generovat?), ale třeba nás to aspoň motivuje implementovat věci ve webu a ne úplně ad hoc jinde (kontaktníček, pozvánky pro sous, …) :-D
5883a5cd28
into master 1 week ago5883a5cd28
.