políčko pro zpětnou vazbu #3

Merged
zelvuska merged 3 commits from zpetna_vazba into master 2 years ago
karelb commented 2 years ago
Owner
There is no content yet.
karelb added 3 commits 2 years ago
zelvuska reviewed 2 years ago
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)')
Poster
Owner

K tomuto podle mě není vytvořená migrace…

K tomuto podle mě není vytvořená migrace…
karelb force-pushed zpetna_vazba from d5a2e26e95 to fdd93aa278 2 years ago
ledoian reviewed 2 years ago
}
input {
input, textarea {
Poster
Owner

Vůbec si nejsem jistý, že se input a textarea má vždycky a všude chovat stejně, udělal bych na to vlastní blok.

Also: co to dělá s korekturovátkem?

Vůbec si nejsem jistý, že se `input` a `textarea` má vždycky a všude chovat stejně, udělal bych na to vlastní blok. Also: co to dělá s korekturovátkem?
ledoian reviewed 2 years ago
for form in formset:
hodnoceni = m.Hodnoceni(
reseni=reseni,
**form.cleaned_data,
Poster
Owner

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“)

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“)
karelb force-pushed zpetna_vazba from fdd93aa278 to e856859ffb 2 years ago
ledoian reviewed 2 years ago
{
Poster
Owner

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…

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…
ledoian reviewed 2 years ago
ledoian left a comment

LGTM

zelvuska merged commit 2c9d283a64 into master 2 years ago
zelvuska deleted branch zpetna_vazba 2 years ago
Owner

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/…)

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/…)
The pull request has been merged as 2c9d283a64.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.