políčko pro zpětnou vazbu #3
Merged
zelvuska
merged 3 commits from zpetna_vazba
into master
2 years ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'zpetna_vazba'
Deleting a branch is permanent. It CANNOT be undone. Continue?
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…
d5a2e26e95
tofdd93aa278
2 years ago}
input {
input, textarea {
Vůbec si nejsem jistý, že se
input
atextarea
má vždycky a všude chovat stejně, udělal bych na to vlastní blok.Also: co to dělá s korekturovátkem?
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“)
fdd93aa278
toe856859ffb
2 years ago{
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
2c9d283a64
into master 2 years agoJeš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/…)
2c9d283a64
.