Skip to content
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

Merged
merged 2 commits into from
Mar 9, 2023
Merged

Vidnost nalog #289

merged 2 commits into from
Mar 9, 2023

Conversation

schrjako
Copy link
Collaborator

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

@schrjako
Copy link
Collaborator Author

Closes #266

Copy link
Collaborator

@matijapretnar matijapretnar left a 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.

@matijapretnar
Copy link
Collaborator

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
Copy link
Collaborator Author

schrjako commented Mar 8, 2023

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

@matijapretnar
Copy link
Collaborator

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 - Infrastruktura za prevode #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
Copy link
Collaborator Author

schrjako commented Mar 8, 2023

Se strinjam da se lahlo PR sprejme.

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

@matijapretnar
Copy link
Collaborator

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

@schrjako
Copy link
Collaborator Author

schrjako commented Mar 8, 2023

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
Copy link
Collaborator

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
Copy link
Collaborator Author

schrjako commented Mar 8, 2023

Je smiselno, imate prav.

@matijapretnar matijapretnar merged commit a9fd3e8 into master Mar 9, 2023
@matijapretnar matijapretnar deleted the vidnost_nalog branch March 9, 2023 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants