Přidána příloha s opravou řešení #113

Merged
zelvuska merged 3 commits from feat/priloha-s-opravou into master 2026-02-21 18:20:35 +01:00
Owner
No description provided.
ticvac left a comment
Owner

Vidím že tam je nějaké notifikace... ale úplně tomu nerozumím, jak se posílá email účastníkovi o opraveném řešení?

A kde předně si je má řešitel stáhnout?

Vidím že tam je nějaké notifikace... ale úplně tomu nerozumím, jak se posílá email účastníkovi o opraveném řešení? A kde předně si je má řešitel stáhnout?
@ -220,0 +232,4 @@
reseni = models.ForeignKey(Reseni, verbose_name='řešení', related_name='opravene_prilohy',
on_delete=models.CASCADE)
vytvoreno = models.DateTimeField('vytvořeno', default=timezone.now, blank=True, editable=False)
Owner

na co je tady blank=True?
auto_now dela podle me přesně to co chceme ne?

na co je tady blank=True? auto_now dela podle me přesně to co chceme ne?
Author
Owner

blank=True je asi celkem jedno.

Pokud jsem to správně pochopil, auto_now by pokaždé, co by byl objekt uložen, uložil aktuální datum a čas. Což zní špatně pro „vytvořeno“.

`blank=True` je asi celkem jedno. Pokud jsem to správně pochopil, `auto_now` by pokaždé, co by byl objekt uložen, uložil aktuální datum a čas. Což zní špatně pro „vytvořeno“.
Owner

chtel jsem napat auto_now_add ;)

chtel jsem napat auto_now_add ;)
zelvuska marked this conversation as resolved
@ -81,1 +82,4 @@
<p>Žádné přílohy</p>
{% endif %}
<h3>Opravené soubory:</h3>
Owner

je tahle cela sekce potreba?

stejně jako s hodnoceními, taky jsou zobraceny pouze v form, kde se i edituji...

je tahle cela sekce potreba? stejně jako s hodnoceními, taky jsou zobraceny pouze v form, kde se i edituji...
Author
Owner

Je potřeba pro řešitele, ale asi má být okolo if not edit.

Je potřeba pro řešitele, ale asi má být okolo `if not edit`.
Owner

jo to me zmátlo... ze se pro orgy I resitele pouziva stejnej template

jo to me zmátlo... ze se pro orgy I resitele pouziva stejnej template
zelvuska marked this conversation as resolved
Owner

jo už to vidím...
jen proklik na detail řešení pres to datum doporučeni není moc intuitivní... jako řešitel jsem tu detailní stránku nikdy neviděl... ups

jo už to vidím... jen proklik na detail řešení pres to datum doporučeni není moc intuitivní... jako řešitel jsem tu detailní stránku nikdy neviděl... ups
Owner

z hlediska kódu za me dobry...

pokud nechceme udělat neco s tím UX ze strany řešitele?
(to ze kliknuti na datum v tabulce zobrazí detail...)
tak bych to mergnul

z hlediska kódu za me dobry... pokud nechceme udělat neco s tím UX ze strany řešitele? (to ze kliknuti na datum v tabulce zobrazí detail...) tak bych to mergnul
Author
Owner

To mi nepřijde jako scope tohoto pull requestu. (A alespoň v e-mailu chodí přímo odkaz.)

To mi nepřijde jako scope tohoto pull requestu. (A alespoň v e-mailu chodí přímo odkaz.)
zelvuska merged commit 3f2d59a247 into master 2026-02-21 18:20:35 +01:00
zelvuska deleted branch feat/priloha-s-opravou 2026-02-21 18:20:36 +01:00
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!113
No description provided.