LEdoian ledoian · it/its
ledoian commented on pull request mam/mamweb#83 2025-01-14 22:08:47 +01:00
odevzdavatko: odesílání emailu řešiteli při změně zpětné vazby

A můj přístup by byl z models.py default zrušit úplně a naopak nastavit null=False. To vynutí, že ta tabulka bude vždycky chtít explicitní nastavení při přidání řádku a…

ledoian commented on pull request mam/mamweb#83 2025-01-14 22:02:49 +01:00
odevzdavatko: odesílání emailu řešiteli při změně zpětné vazby

Pokud chceme být hodně explicitní, tak můžeme nastavit default=True a pak napsat něco typu Resitel.objects.update(notifikace_o_zpetne_vazbe=False), ale to mi přijde zbytečně…

ledoian commented on pull request mam/mamweb#83 2025-01-14 22:01:50 +01:00
odevzdavatko: odesílání emailu řešiteli při změně zpětné vazby

Tak do tabulky se tak jak tak uloží ten boolean, který přijde ve formuláři, tedy pokud má formulář initial=True (což iirc má), tak se tady default použije jen pro stávající…

ledoian commented on pull request mam/mamweb#81 2025-01-14 21:58:08 +01:00
Admin strxfrm mac

Proto navrhuji ….error(f"…{e}…"). A přijde mi, že stejně v ≥80% případů při chybě budu potřebovat koukat do kódu, takže hlavně potřebuji umět najít místo v kódu…

ledoian commented on pull request mam/mamweb#83 2025-01-14 21:48:08 +01:00
odevzdavatko: odesílání emailu řešiteli při změně zpětné vazby

Však taky komentuji jen migraci a ne formulář – myslím, že máme na mysli to samé :-)

ledoian commented on pull request mam/mamweb#83 2025-01-14 20:09:21 +01:00
odevzdavatko: odesílání emailu řešiteli při změně zpětné vazby

A ještě jeden komentář, spíš do budoucna: Asi budeme někdy chtít mít vyloženě nějaká uživatelská nastavení (posílání notifikací, výchozí filtr v odevzdávátkové tabulce,…

ledoian commented on pull request mam/mamweb#81 2025-01-14 19:54:30 +01:00
Admin strxfrm mac

Aspoň u mně tohle vygeneruje něco typu 2025-01-14 19:46:39,302 - mamweb.admin - ERROR - OSError: Invalid argument a nic víc, což není moc užitečné (nikde není odkaz přímo do zdrojáku ani chybová hláška nejde najít ve zdrojácích). Navrhuji něco typu logger.error(f"Zdá se, že nefunguje strxfrm: {e}; admin bude špatně seřazený.")

ledoian commented on pull request mam/mamweb#83 2025-01-14 19:41:32 +01:00
odevzdavatko: odesílání emailu řešiteli při změně zpětné vazby

Až na ty dvě drobnosti za mě taky OK.

ledoian commented on pull request mam/mamweb#83 2025-01-14 19:41:05 +01:00
odevzdavatko: odesílání emailu řešiteli při změně zpětné vazby

Ideálně asi pojmenovat jinak, tohle potenciálně není jediné upozornění, které chceme posílat. Navrhuji upozornovat_na_opravy_reseni

ledoian commented on pull request mam/mamweb#83 2025-01-14 19:40:07 +01:00
odevzdavatko: odesílání emailu řešiteli při změně zpětné vazby

Za mě spíš default=False, je to změna chování webu a těch mailů potenciálně může být docela hodně… Ale je to asi otázka spíš mého osobního názoru (věci nemají random měnit chování a vývojáři nemají diktovat uživatelům, co chtějí) než něčeho striktního, klidně mě přehlasujte…

ledoian created pull request mam/mamweb#82 2024-12-03 21:28:55 +01:00
Vnořené rámečky mají být vidět
ledoian pushed to nested_ramecky at mam/mamweb 2024-12-03 21:28:34 +01:00
497bb054ee Vnitřní rámečky by měly jít vidět
ledoian created branch nested_ramecky in mam/mamweb 2024-12-03 21:28:33 +01:00
ledoian commented on pull request mam/mamweb#80 2024-12-03 20:45:03 +01:00
Práva v data/* a načítané pomocí ./manage.py loaddata

Možná by ta práva měla existovat :-D A taky pak můžeš smazat various/management/commands/save_org_permissions.py

ledoian commented on pull request mam/mamweb#81 2024-12-03 20:26:13 +01:00
Admin strxfrm mac

Prosím vypiš někde varování (logger.warning("...")), že se to děje, protože na tenhle kód se pravděpodobně zapomene a s varováním to aspoň půjde dohledat…

ledoian commented on pull request mam/mamweb#78 2024-11-27 17:02:40 +01:00
WIP: Dokumentace: úvod k nástrojům pro vývoj

Možná nějaké overview na začátek? Ale to se asi pozná, až se to napíše celé a jak moc to bude vypadat overwhelming…

ledoian commented on pull request mam/mamweb#78 2024-11-27 17:00:31 +01:00
WIP: Dokumentace: úvod k nástrojům pro vývoj

pip!

ledoian commented on pull request mam/mamweb#78 2024-11-26 22:19:19 +01:00
WIP: Dokumentace: úvod k nástrojům pro vývoj

A ještě dokumentace ve Sphinxu

ledoian commented on pull request mam/mamweb#78 2024-11-26 19:02:58 +01:00
WIP: Dokumentace: úvod k nástrojům pro vývoj

Ještě jak zavřít vim a nastavit editor na nano + jak použít nano :-)

ledoian commented on pull request mam/mamweb#78 2024-11-26 19:02:58 +01:00
WIP: Dokumentace: úvod k nástrojům pro vývoj

wtf já nemůžu požadovat změny po sobě?