LEdoian ledoian · it/its
ledoian commented on pull request mam/mamweb#87 2025-02-11 22:07:12 +01:00
Přednášky

Osobní preference: mít z toho „klasické“ datové typy (pole dvojic), ne zip objekty, které se konzumují a nedají se proto cyklit opakovaně. (Až to někdo bude zkoušet, tak je imho velká šance, že mu to bude dělat blbosti a nebude chápat proč…)

ledoian commented on pull request mam/mamweb#87 2025-02-11 22:07:12 +01:00
Přednášky

A neumíme v takovém případě dát rozumnou hlášku na frontend? (Jakože, špatná sada přednášek je asi divný okrajový případ a mám trochu chuť seznamu o kterém se aktuálně hlasuje zakázat změny, ale je otázka, jestli nemůže nastat nějaký jiný divný stav…)

ledoian commented on pull request mam/mamweb#87 2025-02-11 22:07:12 +01:00
Přednášky

Mostly LGTM

ledoian commented on pull request mam/mamweb#87 2025-02-11 22:07:12 +01:00
Přednášky

Asi OK. Možná si teoreticky můžeme místo účastníků u všeho držet celý objekt s informacemi o hlasování k celému seznamu, ale to za to nestojí, zvlášť pokud budeme mít ambice zmigrovat to celé na Osoby.

ledoian commented on pull request mam/mamweb#87 2025-02-11 22:07:11 +01:00
Přednášky

Also: tu Osobu (s null=True) si IMHO chceme ukládat už teď, ať pak můžeme zmigrovat a nechybí nám data… (původní hlasovátko ještě nevědělo nic o účastnických uživatelích/osobách…)

ledoian commented on pull request mam/mamweb#87 2025-02-11 22:07:11 +01:00
Přednášky

Navrhuji mít kolem každého bloku hlasování aspoň dummy <div>, ať se kdyžtak dají různě popřeskládat (kupř. nějakým flexem na ultraširokém monitoru vedle sebe, idk) a nehrozí někde rozbití CSS. (Případně ať si můžu nahackovat dodatečná CSS :-D)

ledoian commented on pull request mam/mamweb#87 2025-02-11 22:07:11 +01:00
Přednášky

Hmm, existuje čisté a praktické řešení. To čisté a datazachovávající je mít pro osobu nový FK (s null=True, blank=False) a mít stará řešení se string-účastníkem a nová s Osobou.

ledoian commented on pull request mam/mamweb#87 2025-02-11 21:36:51 +01:00
Přednášky

Also: tu Osobu (s null=True) si IMHO chceme ukládat už teď, ať pak můžeme zmigrovat a nechybí nám data… (původní hlasovátko ještě nevědělo nic o účastnických uživatelích/osob…

ledoian created branch merge-vetev-pro-prednasky in mam/mamweb 2025-02-11 19:58:59 +01:00
ledoian pushed to merge-vetev-pro-prednasky at mam/mamweb 2025-02-11 19:58:59 +01:00
ledoian pushed to stahnout_reseni_jako_zip at mam/mamweb 2025-01-21 23:53:07 +01:00
dd0872cc83 Inicialzujeme osy na správnou sadu objektů, snad
ledoian commented on pull request mam/mamweb#85 2025-01-21 22:07:04 +01:00
Zpřístupnění informací z "jak se o nás dozvěděli" propagaci

A commit message by příště mohla být lepší, protože po zamergování už nemá jméno větve žádný význam, naopak commit se někde objevit může…

ledoian commented on pull request mam/mamweb#85 2025-01-21 22:06:17 +01:00
Zpřístupnění informací z "jak se o nás dozvěděli" propagaci

To není to, co znamená „grey“…

ledoian commented on pull request mam/mamweb#85 2025-01-21 22:06:17 +01:00
Zpřístupnění informací z "jak se o nás dozvěděli" propagaci

Tohle je úplně jiný vzhled, než máme asi úplně všude…

ledoian commented on pull request mam/mamweb#85 2025-01-21 22:06:17 +01:00
Zpřístupnění informací z "jak se o nás dozvěděli" propagaci

Mám pocit, že tenhle padding divně odsazuje nadpisy a to nezarovnání se mi subjektivně dost nelíbí.

ledoian commented on pull request mam/mamweb#85 2025-01-21 22:06:17 +01:00
Zpřístupnění informací z "jak se o nás dozvěděli" propagaci

Tohle strašně moc vypadá jako tabulka, nebylo by lepší to skutečně mít udělané jako tabulku? (Nepamatuji si, co flex umí a tabulka ne, takže je možné, že máme nějaký důvod to tak neudělat…)

ledoian created pull request mam/mamweb#84 2025-01-15 01:20:05 +01:00
WIP: Stáhnout řešení jako ZIP
ledoian created branch stahnout_reseni_jako_zip in mam/mamweb 2025-01-15 01:19:05 +01:00
ledoian pushed to stahnout_reseni_jako_zip at mam/mamweb 2025-01-15 01:19:05 +01:00
cd914f4524 Hrubá implementace stahování jako ZIP