search

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

Sort institutions alphabetically #235

Closed jureslak closed 4 years ago

jureslak commented 4 years ago

Closes #232.

matijapretnar commented 4 years ago

Zakaj pa ne bi order_by uporabljali kot povsod drugod?

jureslak commented 4 years ago

Ker sem to naredil med vajami iz NM, pa nisem bil pri sebi, zato je pa draft :)

On Fri, Nov 29, 2019, 08:38 Matija Pretnar notifications@github.com wrote:

Zakaj pa ne bi order_by uporabljali kot povsod drugod?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ul-fmf/projekt-tomo/pull/235?email_source=notifications&email_token=AAKUPHWFJMFXD3WHEKTINRTQWDBGBA5CNFSM4JSUFAYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFODJ7A#issuecomment-559691004, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKUPHUAYSFEDHSW22TTYZTQWDBGBANCNFSM4JSUFAYA .

jureslak commented 4 years ago

Zdi se mi, da je baza nastavljena tako, da je locale angleški, kar pomeni, da sortiranje ne dela prav.

jaanos commented 4 years ago

Za ustrezno urejanje se lahko uporabi ICU - bodisi kot razširitev baze, ali pa v Pythonu s pyicu, glej ul-fmf/urnik#78.

matijapretnar commented 4 years ago

Ne, to je samo na urniku, ki še vedno teče na SQLite. Tomo teče na Postgresu in bi moral stvari sortirati pravilno.

matijapretnar commented 4 years ago

Ni idealno, še vedno je Škofijska med Seminar in Srednja šola, ampak za začetek bo. Tega ne bi reševal z uporabo ICU v kodi, temveč s pravilno nastavljenim Postgresom.

jaanos commented 4 years ago

Aja, če je Postgres zadaj, potem ni problema, samo pravi collation je treba nastavit (najbolje kar na nivoju baze).