-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Vidnost nalog #289
Vidnost nalog #289
Conversation
Closes #266 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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