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

Fixed double importing and an incorrect weekday range #98

Closed tinaZwittnig closed 3 years ago

tinaZwittnig commented 3 years ago

V project/settings/production.py je izbrisan dvojni importing 

import ldap
from django_auth_ldap.config import LDAPSearch

ki se zgodi v 1. in 2. vrstici in v 31.,32. vrstici.

Prav tako je odpravljen bug, ki ne prikazuje petkov v specifičnem tednu. 

veljavni_dnevi = [dan for dan in range(0, 5) if semester.od <= teden + datetime.timedelta(days=dan) <= semester.do]

Dnevi so v modelih šteti od 1->ponedeljek ... 5->petek , tako da je treba v range zanki iterirati do 6.

tinaZwittnig commented 3 years ago

Popravljen hrošč ki ne prikazuje srečanj, ki se ne prikazujejo.

jureslak commented 3 years ago

Ta popravek je super, samo daj lepo v opisu PRja (lahko ga urediš) opiši kaj je problem, kako se je kazal na urniku in zakaj zdaj deluje prav.

Uredi commit message in pull request v nekaj bolj razumljivega, popravi diff kot je rekel Matija, pa bo.

tinaZwittnig commented 3 years ago

Pozdravljena, sem popravila commit message, nevem pa a je že tako kot mora biti? Hvala za pomoč!

jureslak commented 3 years ago

Popravila si opis PR-ja, ne commit message.

Najlepše bi bilo, da commit (ne PR!) razdelis na dva dela, v prvem odpraviš dvojni import, v drugem pa popraviš razpon dni v tednu.

Commit message (za drugi commit) bi bil lahko recimo "Fix incorrect weekday range" ali pa kaj takega, ker niso res petki jedro problema. Tudi to da vključujemo nedelje je narobe.

Ce se ti ne da razdeliti commitov pa vseeno vsaj amendaj tale message.

On Thu, Aug 27, 2020, 13:34 TinaZwittnig notifications@github.com wrote:

Pozdravljena, sem popravila commit message, nevem pa a je že tako kot mora biti? Hvala za pomoč!

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ul-fmf/urnik/pull/98#issuecomment-681892515, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAKUPHWUY5ZHWLSMKOQBOY3SCZADTANCNFSM4QHIJT2Q .