search

ul-fmf / urnik

Spletna storitev za prikaz in urejanje urnika
https://urnik.fmf.uni-lj.si/
MIT License
8 stars 10 forks source link

Izbira semestra premaknjena v URL #75

Closed jaanos closed 4 years ago

jaanos commented 4 years ago

Kot je bilo omenjeno v #70 in #72, bi bilo bolje, da je izbira semestra eksplicitna v URL-ju. Ta PR doda možnost dodajanja semester/<int:semester_id>/ pred relevantne URL-je - če te predpone ni, potem se URL nanaša na trenutni semester (semester_id == None).

Ne zagotavljam, da sem ujel čisto vsako povezavo, da bo pravilno kazala na izbrani semester - urejanja urnika in rezervacij se nisem dotikal. Poleg tega nisem čisto prepričan, če je smiselno, da imamo Proste učilnice za prejšnje semestre - konkretno v tem primeru trenutno ni možno izbirati tedna za prikaz, saj se prikažejo samo prihajajoči tedni.

Po mojem lahko v tem pull requestu uredimo še ostale pripombe iz #70.

Closes #70.

jaanos commented 4 years ago

Sem zdaj onemogočil rezervacije in pregledovanje prostih učilnic pri pregledovanju starih urnikov. Proste učilnice sicer še vedno delujejo tudi za prejšnje semestre, samo povezave do tega pogleda ni nikjer.

CSS in postavitev povezave na trenutni urnik pa prepuščam @jureslak.

jureslak commented 4 years ago

image

Closes #70.

Sem dodal stil in popravil neveljaven html pri seznamu urnikov. Tega sem tudi obrnil okrog, da je novi urnik na vrhu in da so stari urniki čedalje bolj v pozabi. Malo sem klikal okrog in vse zgleda kot da deluje, tako da approvam predajam @matijapretnar da merga, če je ok.