ul-fmf / projekt-tomo

Spletna storitev za poučevanje programiranja
https://www.projekt-tomo.si
GNU Affero General Public License v3.0
14 stars 23 forks source link

Vidnost nalog #289

Closed schrjako closed 1 year ago

schrjako commented 1 year ago

Manjkajo se prevodi dolocenih dodanih besedil, vendar ne vem kako jih dodati. Tezavo opisal v #266

schrjako commented 1 year ago

Closes #266

matijapretnar commented 1 year ago

Tisti del, s katerim sem se strinjal, sem že dal na master. Vprašanje je, kaj naredimo s prevodi - to je po mojem ločen issue?

schrjako commented 1 year ago

Ali ne bi bilo smiselno, da prevodi stvari iz tega pr-ja so v tem pr-ju?

matijapretnar commented 1 year ago

Ja, se strinjam. Težava je, da že nekaj časa nisem delal s prevodi in ne vem, ali je vzrok težave s prevodi:

  1. sistemski, torej da infrastruktura za prevode ne dela pravilno
  2. v tem, da ste v tem PRju nekaj napisali narobe. V vsakem primeru je treba odpreti ločen issue z dokumentacijo prevodov, … (to delam ravnokar - #290), v primeru 2. pa je poleg tega je pa treba ugotoviti, ali ta PR potrebuje še kaj dodatnega dela ali ne.

Glede na to, da so prevodi trenutno itak bolj zmedeni, to ne bi smelo ustaviti PRja, zato predlagam, da se ga sprejme, ko se pobriše vidnost nalog iz datotek za urejanje (ali me prepriča nasprotno). Še to, kakšna je logika pri commitu Add not visible problems to problem_set statistics - je to fixup prvotnega PRja ali kaj drugega?

schrjako commented 1 year ago

Se strinjam da se lahlo PR sprejme.

In ja, tisti commit doda skrite probleme nazaj v statistiko, ker so bili na zacetku izkljuceni.

matijapretnar commented 1 year ago

Kaj pa naredimo s prikazom in urejanjem vidnosti v datotekah? Kakšni zadržki, da to damo ven?

schrjako commented 1 year ago

Meni se zdi smiselno, da bi se vidnost naloge videlo tudi v datotekah. Moram pa priznat da ne poznam tomo-workflowa kot ucitelj sploh in v bistvu nimam pojma, kaj je tle pravi odgovor.

matijapretnar commented 1 year ago

Sem še malo razmislil. Ko bomo enkrat naredili #286, bo vidnost na ProblemInstance, datoteke za urejanje pa na Problem, tako da bomo takrat to tako ali tako morali dati ven.

schrjako commented 1 year ago

Je smiselno, imate prav.