Pavlova verze URLconfu #33

Closed
ledoian wants to merge 2 commits from vyroci-fix into vyroci
ledoian commented 1 year ago
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 1 year ago
8d386d0289 Pavlova verze URLconfu
Poster
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 1 year ago
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.