Pavlova verze URLconfu #33

Closed
ledoian wants to merge 2 commits from vyroci-fix into vyroci
Owner

Upravil jsem u toho i komentáře kolem a reverzní cestu, nic by na ni nemělo vést, tak jsem si to rovnou dovolil.

Uznávám, že tenhle commit dělá víc změn dohromady, tak se kdyžtak použije jen parciálně…

Nejsem si jistý, co dělá pull-request na pull-request, kdyžtak se dá mergenout ten předchozí (cílový) a pak aplikovat patch z tohohle.

Upravil jsem u toho i komentáře kolem a reverzní cestu, nic by na ni nemělo vést, tak jsem si to rovnou dovolil. Uznávám, že tenhle commit dělá víc změn dohromady, tak se kdyžtak použije jen parciálně… Nejsem si jistý, co dělá pull-request na pull-request, kdyžtak se dá mergenout ten předchozí (cílový) a pak aplikovat patch z tohohle.
ledoian added 2 commits 2023-06-13 23:24:11 +02:00
Upravil jsem u toho i komentáře kolem a reverzní cestu, nic by na ni
nemělo vést, tak jsem si to rovnou dovolil.

Uznávám, že tenhle commit dělá víc změn dohromady, tak se kdyžtak
použije jen parciálně…
Author
Owner

Dobře, klidně si sám odhalím bug: teď můžou vznikat cesty /sraz-30let/ a (výhledově, možná, hypoteticky) /sraz/nebeska-rybna-2023/, což se liší jen málo, takže to taky může být trochu matoucí. Ale tenhle orgosraz uvidí jen orgové podle odkazu a teď, takže na to nenarazí, účastníci na to taky nenarazí a webaři si můžou záznam z mamweb/urls.py schovat dostatečně nízko v tom souboru, aby je to taky nemátlo.

Jediné riziko asi je, že se stávající org v budoucnu pokusí tipnout si adresu, podle mého hypotetického vzoru účastnických srazů jako /sraz/30let/ a nenajde to, a protože neměl uloženou záložku, tak správnou adresu bude muset dohledat od někoho jiného.

(Dobře, větší riziko je, že se přepíše web a rozbijou se odkazy nebo se stránka o výročí dokonce úplně ztratí. Ale zatím jsem idealista :-))

Dobře, klidně si sám odhalím bug: teď můžou vznikat cesty `/sraz-30let/` a (výhledově, možná, hypoteticky) `/sraz/nebeska-rybna-2023/`, což se liší jen málo, takže to taky může být trochu matoucí. Ale tenhle orgosraz uvidí jen orgové podle odkazu a teď, takže na to nenarazí, účastníci na to taky nenarazí a webaři si můžou záznam z `mamweb/urls.py` schovat dostatečně nízko v tom souboru, aby je to taky nemátlo. Jediné riziko asi je, že se stávající org v budoucnu pokusí tipnout si adresu, podle mého hypotetického vzoru účastnických srazů jako `/sraz/30let/` a nenajde to, a protože neměl uloženou záložku, tak správnou adresu bude muset dohledat od někoho jiného. (Dobře, větší riziko je, že se přepíše web a rozbijou se odkazy nebo se stránka o výročí dokonce úplně ztratí. Ale zatím jsem idealista :-))
Owner

Mě přijde, že dává smysl mít spíš složku srazy (ať už srazy nebo sraz), a pod tím mít prostě různé srazy…

Mě přijde, že dává smysl mít spíš složku srazy (ať už `srazy` nebo `sraz`), a pod tím mít prostě různé srazy…
Owner

Jak moc trváš na tomto pullrequestu?

Jak moc trváš na tomto pullrequestu?
Owner

Vyřešeno v #32.

Vyřešeno v #32.
zelvuska closed this pull request 2023-06-15 16:56:00 +02:00

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: mam/mamweb#33
No description provided.