LEdoian ledoian · it/its
ledoian created pull request mam/mamweb#33 2023-06-13 23:24:10 +02:00
Pavlova verze URLconfu
ledoian pushed to vyroci-fix at mam/mamweb 2023-06-13 23:21:36 +02:00
96a3db10e7 Fix typo
ledoian pushed to vyroci-fix at mam/mamweb 2023-06-13 23:17:38 +02:00
8d386d0289 Pavlova verze URLconfu
ledoian created branch vyroci-fix in mam/mamweb 2023-06-13 23:17:38 +02:00
ledoian commented on pull request mam/mamweb#32 2023-06-13 23:13:30 +02:00
Výroční sraz M&M

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 on pull request mam/mamweb#32 2023-06-13 23:12:17 +02:00
Výroční sraz M&M

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ů.

ledoian commented on pull request mam/mamweb#32 2023-06-13 22:41:31 +02:00
Výroční sraz M&M

Klidně, když se na to nezapomene (nebo když těm dvěma někdo řekne, že se mají prostě normálně zaregistrovat), spíš mi jde o to, aby tady nebyli nahardcodění, protože „určitě…

ledoian commented on pull request mam/mamweb#32 2023-06-13 19:12:16 +02:00
Výroční sraz M&M

Tyhle fieldy vůbec nemusejí být krátké. Zvlášť dodat je potenciálně dlouhé a může mít i víc, než 256 znaků. (Pro srovnání, tenhle komentář sám o sobě má 188 znaků, resp. 214 bytů v UTF-8.)

ledoian approved mam/mamweb#32 2023-06-13 19:12:16 +02:00
Výroční sraz M&M

Jestli to funguje, tak vesměs LGTM. Dvě věci bych změnil, totiž délku textových polí (zkopírovat třeba z poznámek z korekturovátka), a schování adres do separátní složky, ať nejsou v kořeni…

ledoian commented on pull request mam/mamweb#32 2023-06-13 19:12:16 +02:00
Výroční sraz M&M

Tohle funguje? Měl jsem za to, že ListView vyrábí v kontextu object_list… (A možná bych to malinko preferoval, kvůli konzistenci; byť by asi rozumné naopak bylo plošně všechny object_list přepsat na to, seznamy čeho to reálně jsou. Asi spíš výhledově, do Kanboardu…)

ledoian commented on pull request mam/mamweb#32 2023-06-13 19:12:16 +02:00
Výroční sraz M&M

Template prohlížím jen zběžně, pokud to dobře vypadá (i ve Chrome a Edgi :-P), tak je mi asi jedno, jak to vypadá…

ledoian commented on pull request mam/mamweb#32 2023-06-13 19:12:16 +02:00
Výroční sraz M&M

Non-field errors bych čekal spíš nahoře než dole…

ledoian commented on pull request mam/mamweb#32 2023-06-13 19:12:16 +02:00
Výroční sraz M&M

Nevím, jaká je dohoda, ale možná je spíš žádoucí, aby byli všichni účastníci v databázi, protože jinak se na Káťu a (R)adima bude snadno zapomínat a nebude to vidět.

ledoian commented on pull request mam/mamweb#32 2023-06-13 19:12:16 +02:00
Výroční sraz M&M

Jo, a tenhle komentář se spíš týká modelu než migrace, ale podruhé se mi ho psát nechce :-) (103 znaků)

ledoian commented on pull request mam/mamweb#32 2023-06-13 19:12:16 +02:00
Výroční sraz M&M

Aha, to je jen podmnožina fieldů. OK then…

ledoian commented on pull request mam/mamweb#32 2023-06-13 19:12:15 +02:00
Výroční sraz M&M

Neumí se tohle nějak odkázat na vlastní fieldy? Čekal bych tam nějaké self.fields. (Ale asi je to dost šumák, stejně se to jednou napíše, jednou použije a pak jednou možná smaže, takže je to spíš možnost se naučit nové triky s Djangem než potřeba upravit…)

ledoian commented on pull request mam/mamweb#32 2023-06-13 19:12:15 +02:00
Výroční sraz M&M

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)

ledoian commented on pull request mam/mamweb#32 2023-06-13 18:59:57 +02:00
Výroční sraz M&M

Jo, a tenhle komentář se spíš týká modelu než migrace, ale podruhé se mi ho psát nechce :-) (103 znaků)

ledoian commented on pull request mam/mamweb#32 2023-06-13 18:55:55 +02:00
Výroční sraz M&M

Aha, to je jen podmnožina fieldů. OK then…

ledoian pushed to ledoian at mam/mamweb 2023-06-03 14:55:23 +02:00
60c3c1b34b Zrušení nepotřebností v .gitignore _NA MÉ VĚTVI_.
d6697e0f08 Moje soukromé TODO
Compare 2 commits »