apluslms / gitmanager

Git manager service for A+. Separated from the MOOC-Grader.
0 stars 6 forks source link

Disallow str in dates (AnyDate) #17

Open lainets opened 2 years ago

lainets commented 2 years ago

https://github.com/apluslms/gitmanager/blob/ba22bed9f1e0e7d39c0e5d937e676cda328cf8b4/access/course.py#L325-L327 should not allow str, as said in the comment. Not sure when the Undefined type issue occurs as removing the str didn't cause any crashes for me.

Invalid dates can be passed to A+ as long as the str exists. Fortunately, the issue is noticed by A+ but the error message will be much less clear when reported by A+ instead of Git Manager.