Conversation
|
Closes #266 |
matijapretnar
left a comment
There was a problem hiding this comment.
Jaz bi prikaz in urejanje vidnosti nalog imel samo v spletnem vmesniku, v datotekah pa ne, sicer bo zmeda.
fa8655e to
61fcbd0
Compare
|
Tisti del, s katerim sem se strinjal, sem že dal na |
|
Ali ne bi bilo smiselno, da prevodi stvari iz tega pr-ja so v tem pr-ju? |
|
Ja, se strinjam. Težava je, da že nekaj časa nisem delal s prevodi in ne vem, ali je vzrok težave s prevodi:
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? |
|
Se strinjam da se lahlo PR sprejme. In ja, tisti commit doda skrite probleme nazaj v statistiko, ker so bili na zacetku izkljuceni. |
|
Kaj pa naredimo s prikazom in urejanjem vidnosti v datotekah? Kakšni zadržki, da to damo ven? |
|
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. |
|
Sem še malo razmislil. Ko bomo enkrat naredili #286, bo vidnost na |
|
Je smiselno, imate prav. |
61fcbd0 to
bd6340a
Compare
Manjkajo se prevodi dolocenih dodanih besedil, vendar ne vem kako jih dodati. Tezavo opisal v #266