políčko pro zpětnou vazbu #3
Loading…
Reference in a new issue
No description provided.
Delete branch "zpetna_vazba"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
@ -113,6 +113,8 @@ class Hodnoceni(bm.SeminarModelBase):problem = models.ForeignKey(am.Problem, verbose_name='problém',related_name='hodnoceni', on_delete=models.PROTECT)feedback = models.CharField('zpětná vazba', max_length=120, blank=True, default='', help_text='Zpětná vazba řešiteli (plain text)')K tomuto podle mě není vytvořená migrace…
d5a2e26e95tofdd93aa278@ -418,3 +418,3 @@}input {input, textarea {Vůbec si nejsem jistý, že se
inputatextareamá vždycky a všude chovat stejně, udělal bych na to vlastní blok.Also: co to dělá s korekturovátkem?
@ -268,1 +262,4 @@for form in formset:hodnoceni = m.Hodnoceni(reseni=reseni,**form.cleaned_data,Tady bych byl spíš pro být explicitní. Tohle totiž jinak znemožní do toho formuláře cokoliv dalšího přidat (kupř. mě napadá zaškrtávátko „poslat e-mail se zpětnou vazbou“)
fdd93aa278toe856859ffb@ -285,6 +282,7 @@ class ResitelReseniView(DetailView):{Není tady jednodušší rovnou vracet ten objekt toho hodnocení? Nesouvisí přímo s aktuálními změnami, ale když už to vidím, tak to píšu…
LGTM
Ještě k tomu mám dvě drobné poznámky, které nás určitě časem bude štvát, ale nebrání zamergeování (to jsem nemusel mazat tu větev, co?):
Žere to odstavce. (Déjà vu – korekturovátko.)
A chtělo by to dát řešitelům nějak vědět, že mají nový komentář ke svému řešení, například to napsat v „moje řešení“ (speciálně proto, že jsou tam teď odkazy přes datum, takže někteří řešitelé vůbec nemusí vědět o existenci stránky resitel/reseni/…)