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 +4 -4
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 07c465e6ca - Show all commits

2
mamweb/urls.py
View file

@ -69,7 +69,7 @@ urlpatterns = [
# path('api/', include(router.urls)), # path('api/', include(router.urls)),
# Výroční sraz # Výroční sraz
path('srazy/', include('vyroci.urls')), path('sraz/30-let/', include('vyroci.urls')),
zelvuska marked this conversation as resolved Outdated
ledoian commented 2023-06-13 18:48:16 +02:00
Outdated
Review
Copy link

Nechceš tomu radši vyrobit složku? Přijde mi, že zrovna tohle nemusí úplně plevelit kořenovou cestu, když je to jednorázová akce prakticky bez jakýchkoliv dalších souvislostí. A klidně i jako org-sraz nebo dokonce sraz-30-let, ať nám kdyžtak zbyde /sraz na nějaké častější aktivity (resp. ať až se to zruší nevzniknou falešně oživené odkazy, které ale povedou někam jinam)

Nechceš tomu radši vyrobit složku? Přijde mi, že zrovna tohle nemusí úplně plevelit kořenovou cestu, když je to jednorázová akce prakticky bez jakýchkoliv dalších souvislostí. A klidně i jako `org-sraz` nebo dokonce `sraz-30-let`, ať nám kdyžtak zbyde `/sraz` na nějaké častější aktivity (resp. ať až se to zruší nevzniknou falešně oživené odkazy, které ale povedou někam jinam)
] ]

6
vyroci/urls.py
View file

@ -5,13 +5,13 @@ from .views import VyrociView, VyrociListView
urlpatterns = [ urlpatterns = [
path( path(
'30-let/', '',
VyrociView.as_view(), VyrociView.as_view(),
name='vyrocni_sraz' name='vyrocni_sraz'
), ),
path( path(
'30-let_vysledky/', 'ucastnici/',
ledoian commented 2023-06-13 23:12:17 +02:00
Outdated
Review
Copy link

Ale ne, to jsme se nepochopili… Měl jsem na mysli, že by celý sraz bydlel v nějaké hlubině, typu mam.mff.cuni.cz/sraz-30let/ a tam už by byly jednoduché cesty – adresa bez dalšího suffixu (tj. tady by byl záznam pro prázdný řetězec) by vedla na tu hlavní stránku a pak třeba mam.….cz/sraz-30let/ucastnici by vedlo na seznam přihlášených účastníků.

Ten hlavní point je, že si nezabiješ adresu mam.….cz/<něco užitečného> touhle jednorázovou věcí, protože pokud by se někdo někdy rozhodl, že chce stránky i pro účastnické srazy (mam.….cz/srazy/nebeska-rybna-2023), tak tam tohle bude překážet¹. (Vzhledem k tomu, jak to bastlíme, bych fakt nečekal, že se někdo bude snažit aplikaci vyroci zobecnit pro libovolné další použití)

Navíc možná může být žádoucí, aby stránka téhle akce existovala na stejné adrese ještě dalších 30 let, aby si to mohli orgové připomínat (zvlášť, pokud bychom tam někde měli nějaké další materiály z akce nebo něco podobného).

A taky mi přijde, že i seznam účastníků (IMO to fakt nejsou výsledky) sémanticky patří do „složky“ (stejného segmentu cesty) toho srazu, tedy že mají být někde pod tím, ne „vedle“.

Řetězec sraz-30let se může změnit, případně by asi šlo to vyloženě schovat do něčeho jako /srazy/org30let/, ale pozor na to, že tím by se opět zabetonovala cestová hierarchie (i.e. pokud se někdy někdo rozhodne, že pro případný účastnický sraz chce spíš adresu mam.….cz/sraz/nebeska-rybna-2023, tak pak vzniknou vedle sebe složky /sraz/ i /srazy/ a bude to mást orgy a webaře a nic s tím nepůjde (bez rozbití odkazů) udělat. Pokud první segment cesty bude něco hodně specifického (sraz-30let asi splňuje), tak je malá šance, že to s něčím bude kolidovat a tedy že může vzniknout potřeba to přesouvat.)

A zároveň tím, že to bude mít v hlavních urls.py vynucený vlastní prefix cesty, tak bude zřejmé, odkud se to bere, narozdíl od includu na prázdnou cestu, ke kterým je netriviální dohledat URLconf záznam (a tedy tranzitivně i view).

¹Jo, jsem si vědom toho, že můžeš pak napsat path('', include(…)) do aplikace pro výročí, ale to už je evidentní prasárna a nepůjde se v tom vyznat.

Ale ne, to jsme se nepochopili… Měl jsem na mysli, že by celý sraz bydlel v nějaké hlubině, typu `mam.mff.cuni.cz/sraz-30let/` a tam už by byly jednoduché cesty – adresa bez dalšího suffixu (tj. tady by byl záznam pro prázdný řetězec) by vedla na tu hlavní stránku a pak třeba `mam.….cz/sraz-30let/ucastnici` by vedlo na seznam přihlášených účastníků. Ten hlavní point je, že si nezabiješ adresu `mam.….cz/<něco užitečného>` touhle jednorázovou věcí, protože pokud by se někdo někdy rozhodl, že chce stránky i pro účastnické srazy (`mam.….cz/srazy/nebeska-rybna-2023`), tak tam tohle bude překážet¹. (Vzhledem k tomu, jak to bastlíme, bych fakt nečekal, že se někdo bude snažit aplikaci `vyroci` zobecnit pro libovolné další použití) Navíc možná může být žádoucí, aby stránka téhle akce existovala na stejné adrese ještě dalších 30 let, aby si to mohli orgové připomínat (zvlášť, pokud bychom tam někde měli nějaké další materiály z akce nebo něco podobného). A taky mi přijde, že i seznam účastníků (IMO to fakt nejsou výsledky) sémanticky patří do „složky“ (stejného segmentu cesty) toho srazu, tedy že mají být někde pod tím, ne „vedle“. Řetězec `sraz-30let` se může změnit, případně by asi šlo to vyloženě schovat do něčeho jako `/srazy/org30let/`, ale pozor na to, že tím by se opět zabetonovala cestová hierarchie (i.e. pokud se někdy někdo rozhodne, že pro případný účastnický sraz chce spíš adresu `mam.….cz/sraz/nebeska-rybna-2023`, tak pak vzniknou vedle sebe složky `/sraz/` i `/srazy/` a bude to mást orgy a webaře a nic s tím nepůjde (bez rozbití odkazů) udělat. Pokud první segment cesty bude něco hodně specifického (`sraz-30let` asi splňuje), tak je malá šance, že to s něčím bude kolidovat a tedy že může vzniknout potřeba to přesouvat.) A zároveň tím, že to bude mít v hlavních `urls.py` vynucený vlastní prefix cesty, tak bude zřejmé, odkud se to bere, narozdíl od includu na prázdnou cestu, ke kterým je netriviální dohledat URLconf záznam (a tedy tranzitivně i view). ¹Jo, jsem si vědom toho, že můžeš pak napsat `path('', include(…))` do aplikace pro výročí, ale to už je evidentní prasárna a nepůjde se v tom vyznat.
ledoian commented 2023-06-13 23:13:30 +02:00
Outdated
Review
Copy link

Možná napíšu patch, ono bude zřejmější, jak to myslím, v kódu a ne to tady rozepisovat (byť argumenty zůstanou tady…)

Možná napíšu patch, ono bude zřejmější, jak to myslím, v kódu a ne to tady rozepisovat (byť argumenty zůstanou tady…)
ledoian commented 2023-06-13 23:24:46 +02:00
Outdated
Review
Copy link

!33

!33
org_required(VyrociListView.as_view()), org_required(VyrociListView.as_view()),
name='vyrocni_sraz_vysledky' name='vyrocni_sraz_ucastnici'
), ),
] ]